> 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 terminal tab settings, it was ugly. The use 
of multiple tabs just ate up too much horizontal space.


- Mark


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25309/#review52308
-----------------------------------------------------------


On Sept. 4, 2014, 9:24 a.m., Mark Chu-Carroll wrote:
> 
> -----------------------------------------------------------
> 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 and Maxim Khutornenko.
> 
> 
> Bugs: aurora-672
>     https://issues.apache.org/jira/browse/aurora-672
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Fix output formatting error in "job status".
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/cli/jobs.py 
> ebc22aaa5a8aed311897b3ce9632b6f7175b6080 
>   src/test/python/apache/aurora/client/cli/test_status.py 
> 311fac02af32e0ed687489a2352164effb4dba96 
> 
> Diff: https://reviews.apache.org/r/25309/diff/
> 
> 
> Testing
> -------
> 
> Ran unit tests; added new test cases.
> 
> 
> Thanks,
> 
> Mark Chu-Carroll
> 
>

Reply via email to