Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Joe Smith
> On Oct. 16, 2014, 4:22 p.m., Alex Levenson wrote: > > Sorry all, I may have reported this one too early -- I think the log.info > > statements are going to stderr, in which case anyone parsing the json > > output that's sent to stdout should have no problems. No worries- thanks for kicking t

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Alex Levenson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26828/#review57044 --- Sorry all, I may have reported this one too early -- I think the log

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Kevin Sweeney
--- 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 o

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Joe Smith
> On Oct. 16, 2014, 2:43 p.m., Zameer Manji wrote: > > I feel we will need to revisit the json output in the future to ensure that > > we don't have to do this process for every subcommand. Agreed. - Joe --- This is an automatically ge

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26828/#review57008 --- Ship it! I feel we will need to revisit the json output in the futu

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Joe Smith
> On Oct. 16, 2014, 2:38 p.m., Alex Levenson wrote: > > src/main/python/apache/aurora/client/api/__init__.py, line 107 > > > > > > might want to be consistent with "squelch" and "clean_output"? Choose > > one, or even

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Joe Smith
--- 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 Z

Re: Review Request 26828: Remove info logging when users request json output

2014-10-16 Thread Alex Levenson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26828/#review57004 --- Ship it! src/main/python/apache/aurora/client/api/__init__.py