Re: Review Request 17133: Improve test coverage for TaskAssignerImpl.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17133/#review32369 --- Ship it! Looks good! - Suman Karumuri On Jan. 20, 2014, 10:12

Re: Review Request 17068: Added docs about git setup.

2014-01-21 Thread Suman Karumuri
On Jan. 18, 2014, 12:34 a.m., Jonathan Boulle wrote: docs/contributing.md, lines 7-8 https://reviews.apache.org/r/17068/diff/2/?file=429765#file429765line7 s/setup/set up/ s/configured/configure/ Done. On Jan. 18, 2014, 12:34 a.m., Jonathan Boulle wrote:

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-21 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/#review32376 --- Ship it!

Re: Review Request 17068: Added docs about git setup.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17068/ --- (Updated Jan. 21, 2014, 4:03 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 17051: Add clientv2 implementations of update and list jobs.

2014-01-21 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17051/#review32377 --- ping? - Mark Chu-Carroll On Jan. 17, 2014, 3:05 p.m., Mark

Re: Review Request 17153: Implement restart command for client v2.

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

Re: Review Request 17051: Add clientv2 implementations of update and list jobs.

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

Re: Review Request 17161: Update all dependencies coming from twitter commons and upgrade to guava 15

2014-01-21 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17161/ --- (Updated Jan. 21, 2014, 11:18 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 17042: Add a web interface to the zookeeper instance.

2014-01-21 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17042/#review32198 --- examples/vagrant/provision-zookeeper.sh

Re: Review Request 17161: Update all dependencies coming from twitter commons and upgrade to guava 15

2014-01-21 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17161/#review32457 --- Ship it! Ship It! - Maxim Khutornenko On Jan. 21, 2014, 11:18

Re: Review Request 17153: Implement restart command for client v2.

2014-01-21 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17153/#review32454 --- Mark - want to put some time in my calendar tomorrow to sit down

Re: Review Request 17161: Update all dependencies coming from twitter commons and upgrade to guava 15

2014-01-21 Thread Jake Farrell
On Jan. 21, 2014, 11:32 p.m., Kevin Sweeney wrote: Thanks for taking care of this Bill. Aside: this seems to warrant a linked ticket (reference it on the first line of your commit so that jenkins will comment). Bill Farner wrote: reference it on the first line of your commit