Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-277

Re: Review Request 19411: Add support for global hooks.

2014-03-20 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19411/ --- (Updated March 20, 2014, 7:52 a.m.) Review request for Aurora, Maxim

Review Request 19466: Add batching to the kill and killall commands.

2014-03-20 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19466/ --- Review request for Aurora, Maxim Khutornenko and Brian Wickman. Bugs:

Re: Review Request 19462: [AURORA-282] add gradle target distDeb

2014-03-20 Thread Damien Hardy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19462/ --- (Updated March 20, 2014, 3:43 p.m.) Review request for Aurora and Jake

Re: Review Request 19462: [AURORA-282] add gradle target distDeb

2014-03-20 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19462/#review37895 --- build.gradle https://reviews.apache.org/r/19462/#comment69672

Re: Review Request 19243: Adding min retention interval for task history.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19243/#review37923 --- Ship it! Ship It! - Bill Farner On March 19, 2014, 9:38 p.m.,

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review37924 --- Update or discard? - Bill Farner On March 1, 2014, 1:27 a.m.,

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/#review37925 ---

Re: Review Request 18714: Disable log and initiate shutdown upon log storage failures.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18714/#review37927 --- Ping ping ping - Bill Farner On March 3, 2014, 11:35 p.m., Bill

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/#review37929 ---

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19431/#review37930 --- src/main/python/apache/aurora/client/cli/jobs.py

Re: Review Request 19078: Part 2 of converting from Packages to Metadata.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19078/#review37932 --- I may have missed something, but where did 'all' come from? - Bill

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-20 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19431/#review37933 --- src/main/python/apache/aurora/client/cli/jobs.py

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-20 Thread Bill Farner
On March 20, 2014, 6:27 p.m., Mark Chu-Carroll wrote: src/main/python/apache/aurora/client/cli/jobs.py, line 293 https://reviews.apache.org/r/19431/diff/1/?file=528673#file528673line293 The instance syntax is mandatory for commands that can operate on instances. The parser is

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19431/#review37936 --- src/main/python/apache/aurora/client/cli/jobs.py

Re: Review Request 18714: Disable log and initiate shutdown upon log storage failures.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18714/#review37939 --- Ship it! Ship It! - Suman Karumuri On March 3, 2014, 11:35

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-20 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19431/ --- (Updated March 20, 2014, 2:48 p.m.) Review request for Aurora, Bill Farner and

Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19494/ --- Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Maxim Khutornenko.

Re: Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19494/#review37975 --- I think this still needs some fine tuning. For example: ship-it

Re: Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Bill Farner
On March 20, 2014, 9:16 p.m., Maxim Khutornenko wrote: I think this still needs some fine tuning. For example: ship-it should clear all previous comments from the reviewer and expose other required reviewers who did not post any comments yet. As it stands now neither of my RBs below

Re: Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19494/ --- (Updated March 20, 2014, 10:06 p.m.) Review request for Aurora, Kevin Sweeney,

Re: Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19494/ --- (Updated March 20, 2014, 10:07 p.m.) Review request for Aurora, Kevin Sweeney,

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/ --- (Updated March 20, 2014, 10:22 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 19494: Change missing shipits tool to serve as a dashboard for all outstanding reviews.

2014-03-20 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19494/#review38026 --- Ship it! Ship It! - Kevin Sweeney On March 20, 2014, 3:07 p.m.,

Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19509/ --- Review request for Aurora, Kevin Sweeney, Mark Chu-Carroll, and Brian Wickman.

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/ --- (Updated March 21, 2014, 2:41 a.m.) Review request for Aurora, Kevin Sweeney

Review Request 19519: Allow override of default RB server in shipits tool.

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19519/ --- Review request for Aurora, Suman Karumuri and Maxim Khutornenko. Repository: