Review Request 21205: Bugfix: restart doesn't notify user about invalid max_total_failures option.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21205/ --- Review request for Aurora, David McLaughlin and Suman Karumuri. Bugs:

Re: Review Request 19788: Add a clientv2 version of the e2e test.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19788/ --- (Updated May 7, 2014, 3:47 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/#review42616 --- Ship it! Ship It! - Suman Karumuri On May 8, 2014, 6:54 p.m.,

Re: Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/#review42615 --- - Suman Karumuri On May 8, 2014, 6:54 p.m., Mark Chu-Carroll

Re: Review Request 21136: Add python import order wrapper/commit hook

2014-05-11 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21136/#review42421 --- Ship it! fwiw, lgtm as-is - Joe Smith On May 6, 2014, 4:10

Review Request 21297: Adding UpdateConfig value checks.

2014-05-11 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21297/ --- Review request for Aurora and Brian Wickman. Bugs: AURORA-404

Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/ --- Review request for Aurora, David McLaughlin and Suman Karumuri. Bugs:

Re: Review Request 21142: Avoid sending tasks in SANDBOX_DELETED state to the GC executor.

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21142/ --- (Updated May 7, 2014, 1:24 a.m.) Review request for Aurora and Maxim

Re: Review Request 20394: Add return code to task/process pages

2014-05-11 Thread David McLaughlin
On May 6, 2014, 6:50 p.m., Brian Wickman wrote: david -- should i submit this? i just realized that you likely don't have commit privileges, so i think this is up to me to submit. Yup, that would be cool. I'm not a committer yet. - David

Re: Review Request 21223: Bugfix: make restart command generate a correct error message in some cases.

2014-05-11 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21223/#review42590 --- Ship it!

Re: Review Request 21247: Add config grouping visualisation to job page

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21247/#review42617 --- Ship it!

Review Request 21142: Avoid sending tasks in SANDBOX_DELETED state to the GC executor.

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21142/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-395