Re: Review Request 25309: Fix output formatting error in job status.

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

Re: Review Request 25309: Fix output formatting error in job status.

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

Re: Review Request 25309: Fix output formatting error in job status.

2014-09-04 Thread Mark Chu-Carroll
On Sept. 4, 2014, 11:42 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/cli/jobs.py, line 543 https://reviews.apache.org/r/25309/diff/1-2/?file=675766#file675766line543 Curious, why not using multiple '\t' instead of spacing? Looking at the output with standard

Re: Review Request 25309: Fix output formatting error in job status.

2014-09-04 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25309/#review52311 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 4, 2014, 1:24

Review Request 25309: Fix output formatting error in job status.

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

Re: Review Request 25309: Fix output formatting error in job status.

2014-09-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25309/#review52212 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 3, 2014, 6:17

Re: Review Request 25309: Fix output formatting error in job status.

2014-09-03 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25309/#review52257 --- Ship it! Ship It! - David McLaughlin On Sept. 3, 2014, 6:17