Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28876/#review64791 --- Ship it! Ship It! - Bill Farner On Dec. 9, 2014, 10:32 p.m., Max

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-11 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28876/#review64787 --- Ping, Bill. - Maxim Khutornenko On Dec. 9, 2014, 10:32 p.m., Maxi

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-10 Thread Zameer Manji
> On Dec. 9, 2014, 3:29 p.m., Zameer Manji wrote: > > src/test/python/apache/aurora/client/cli/test_update.py, line 69 > > > > > > Is it at all possible to avoid the use of raw Mock objects? > > Maxim Khutornenko wrote

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28876/#review64677 --- Ship it! Ship It! - Zameer Manji On Dec. 9, 2014, 2:32 p.m., Max

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-10 Thread Maxim Khutornenko
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-10 Thread Zameer Manji
> On Dec. 9, 2014, 3:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clien

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Maxim Khutornenko
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Bill Farner
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Maxim Khutornenko
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Bill Farner
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Maxim Khutornenko
> On Dec. 9, 2014, 11:29 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 384 > > > > > > This solution seems to be papering over the problem we have with output > > in the clie

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28876/#review64460 --- src/main/python/apache/aurora/client/cli/__init__.py

Re: Review Request 28876: Suppressing redundant client command error messaging.

2014-12-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28876/#review64453 --- Ship it! Master (9926af3) is green with this patch. ./build-suppo