Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-27 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/#review32836 --- Ship it! I say ship this, and I imagine more will fall out as we

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-27 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/ --- (Updated Jan. 27, 2014, 1:21 p.m.) Review request for Aurora, Jonathan Boulle

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-27 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/#review32846 --- Ship it! i already shipited two of the previous reviews, so i'm

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-27 Thread Mark Chu-Carroll
On Jan. 25, 2014, 6:54 p.m., Jonathan Boulle wrote: src/main/python/apache/aurora/client/cli/jobs.py, line 378 https://reviews.apache.org/r/17185/diff/1/?file=435145#file435145line378 1sp Sorry, don't know what you mean by this. It's a line continuation, so it's indented 4; and I

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-27 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/#review32852 --- src/main/python/apache/aurora/client/cli/jobs.py

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-25 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/#review32577 --- k - sorry - this review has been sitting completed in my

Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-22 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/ --- Review request for Aurora, Jonathan Boulle and Brian Wickman. Bugs: aurora-53