Re: Review Request 28474: Added manual perf tests for the scheduling pipeline.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28474/#review63119 --- Ship it! Master (ae4d70f) is green with this patch.

Re: Review Request 28451: Modify TRequestsTransport to raise an exception 4xx or 5xx responses.

2014-11-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28451/#review63126 --- Kevin, you were the last comitter to give a ship it. Can you please

Re: Review Request 28474: Added manual perf tests for the scheduling pipeline.

2014-11-26 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28474/#review63130 --- Thanks a lot for breaking ground on this! I've been thinking about

Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/ --- Review request for Aurora and Kevin Sweeney. Bugs: AURORA-939

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/#review63143 --- src/test/python/apache/aurora/executor/common/test_task_info.py

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/#review63144 --- Ship it! Master (ae4d70f) is green with this patch.

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/ --- (Updated Nov. 26, 2014, 2:28 p.m.) Review request for Aurora and Kevin

Re: Review Request 28445: Improving messages in CronJobManager.

2014-11-26 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28445/#review63146 --- Ship it! Ship It! - Kevin Sweeney On Nov. 25, 2014, 10:58 a.m.,

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/#review63148 --- Ship it! Ship It! - Joshua Cohen On Nov. 26, 2014, 10:28 p.m.,

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/#review63149 --- Ship it! Master (ae4d70f) is green with this patch.

Re: Review Request 28483: Improve unbounded ref exception.

2014-11-26 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28483/#review63154 --- Ship it! Ship It! - Kevin Sweeney On Nov. 26, 2014, 2:28 p.m.,

Review Request 28486: Move sandbox initialization into try block.

2014-11-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28486/ --- Review request for Aurora and Kevin Sweeney. Repository: aurora Description

Re: Review Request 28486: Move sandbox initialization into try block.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28486/#review63157 --- Master (ae4d70f) is green with this patch.

Re: Review Request 28486: Move sandbox initialization into try block.

2014-11-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28486/ --- (Updated Nov. 26, 2014, 3:48 p.m.) Review request for Aurora and Kevin

Re: Review Request 28486: Move sandbox initialization into try block.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28486/#review63160 --- Master (ae4d70f) is green with this patch.

Re: Review Request 28411: Adding quota check into replaceCronTemplate rpc.

2014-11-26 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28411/#review63162 --- @ReviewBot retry - Maxim Khutornenko On Nov. 24, 2014, 9:27

Re: Review Request 28411: Adding quota check into replaceCronTemplate rpc.

2014-11-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28411/#review63163 --- This patch does not apply cleanly on master (8d45482), do you need

Re: Review Request 28411: Adding quota check into replaceCronTemplate rpc.

2014-11-26 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28411/ --- (Updated Nov. 27, 2014, 12:36 a.m.) Review request for Aurora and Bill Farner.