----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26828/#review57011 -----------------------------------------------------------
Weird - I'd think we want logging when the user has requested JSON output - as presumably a wrapper tool can 2>/dev/null easier than a user one - Kevin Sweeney On Oct. 16, 2014, 2:40 p.m., Joe Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26828/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2014, 2:40 p.m.) > > > Review request for Aurora, Bill Farner and Zameer Manji. > > > Bugs: AURORA-860 > https://issues.apache.org/jira/browse/AURORA-860 > > > Repository: aurora > > > Description > ------- > > Remove info logging when users request json output > > > Diffs > ----- > > src/main/python/apache/aurora/client/api/__init__.py > d12132f8ee64a390c9b5b4c9ab26b5e4b2b1bb59 > src/main/python/apache/aurora/client/cli/context.py > 4e94a4e3017f3b2ace38d6d8ce68c3c778c8cd0e > src/main/python/apache/aurora/client/cli/jobs.py > 0277cbed4b7eb927d6e5823b4b41d90b366f81d0 > src/test/python/apache/aurora/client/cli/BUILD > d33e86643a59879c115876c98bd1dc19aa7ae61c > src/test/python/apache/aurora/client/cli/test_status.py > c704daec5a6eee73c7092a201b168881853908e8 > > Diff: https://reviews.apache.org/r/26828/diff/ > > > Testing > ------- > > [tw-172-25-132-201 aurora (jsmith/status_json)]$ ./pants > ./src/test/python/apache/aurora/client/cli:job > > > Thanks, > > Joe Smith > >