Re: Review Request 39057: "aurora config read" command

2015-10-12 Thread David McLaughlin
> On Oct. 11, 2015, 8:43 p.m., Stephan Erb wrote: > > src/main/python/apache/aurora/client/cli/config.py, line 75 > > > > > > I find it a little bit confusing that this command is called `read`. > > Isn't the `bind`

Re: Review Request 39057: "aurora config read" command

2015-10-12 Thread Bill Farner
> On Oct. 11, 2015, 1:43 p.m., Stephan Erb wrote: > > src/main/python/apache/aurora/client/cli/config.py, line 75 > > > > > > I find it a little bit confusing that this command is called `read`. > > Isn't the `bind`

Re: Review Request 39057: "aurora config read" command

2015-10-12 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39057/#review102338 --- src/main/python/apache/aurora/client/cli/config.py (lines 84 -

Re: Review Request 39057: "aurora config read" command

2015-10-11 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39057/#review102193 --- src/main/python/apache/aurora/client/cli/config.py (line 75)

Re: Review Request 39057: "aurora config read" command

2015-10-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39057/#review102076 --- Ship it! src/main/python/apache/aurora/client/cli/config.py

Re: Review Request 39057: "aurora config read" command

2015-10-06 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39057/#review101676 --- Ship it! Master (33d7e21) is green with this patch.

Review Request 39057: "aurora config read" command

2015-10-06 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39057/ --- Review request for Aurora, Kevin Sweeney and Zameer Manji. Bugs: AURORA-1504