Re: Review Request 25450: Fix broken "large update" warning.

2014-09-09 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/#review52777 --- Ship it! Ship It! - David McLaughlin On Sept. 9, 2014, 7:43 p.m.

Re: Review Request 25450: Fix broken "large update" warning.

2014-09-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/#review52776 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 9, 2014, 7:43 p.m

Re: Review Request 25450: Fix broken "large update" warning.

2014-09-09 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/ --- (Updated Sept. 9, 2014, 3:43 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 25450: Fix broken "large update" warning.

2014-09-09 Thread Mark Chu-Carroll
> On Sept. 8, 2014, 4:31 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/client/cli/jobs.py, line 655 > > > > > > This feels redundant. Current output already has success message(s): > > > > .

Re: Review Request 25450: Fix broken "large update" warning.

2014-09-09 Thread Mark Chu-Carroll
> On Sept. 8, 2014, 4:31 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/client/cli/jobs.py, line 655 > > > > > > This feels redundant. Current output already has success message(s): > > > > .

Re: Review Request 25450: Fix broken "large update" warning.

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/#review52624 --- src/main/python/apache/aurora/client/cli/jobs.py

Review Request 25450: Fix broken "large update" warning.

2014-09-08 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/ --- Review request for Aurora, David McLaughlin and Maxim Khutornenko. Bugs: aurora