Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Zameer Manji
On March 3, 2015, 3:21 p.m., Zameer Manji wrote: src/test/python/apache/aurora/client/cli/test_supdate.py, line 211 https://reviews.apache.org/r/31710/diff/1/?file=883801#file883801line211 For the calls like this you can do

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/#review75100 --- Ship it! Master (c0b3701) is green with this patch.

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/#review75087 --- Master (c0b3701) is red with this patch.

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Bill Farner
On March 3, 2015, 11:48 p.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/cli/update.py, line 302 https://reviews.apache.org/r/31710/diff/1/?file=883800#file883800line302 You may want to extract an internal method to do this, e.g.: ``` def

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/ --- (Updated March 4, 2015, 12:19 a.m.) Review request for Aurora and Maxim

Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-1165 and

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/#review75083 --- src/test/python/apache/aurora/client/cli/test_supdate.py

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Bill Farner
On March 3, 2015, 11:21 p.m., Zameer Manji wrote: src/test/python/apache/aurora/client/cli/test_supdate.py, line 211 https://reviews.apache.org/r/31710/diff/1/?file=883801#file883801line211 For the calls like this you can do

Re: Review Request 31710: Fix query and timestamp display when running beta-update status.

2015-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31710/#review75089 --- Ship it! src/main/python/apache/aurora/client/cli/update.py