Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/#review104122 --- Master (c20346c) is green with this patch. ./build-support/jenkin

Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/#review104114 --- src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskPro

Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/#review104115 --- src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskPro

Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/ --- (Updated Oct. 26, 2015, 4:16 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/#review104109 --- This patch does not apply cleanly on master (c20346c), do you need

Re: Review Request 39669: Add missing binding for ZooKeeperTestServer when running a local scheduler.

2015-10-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39669/#review104104 --- Ship it! Ship It! - Zameer Manji On Oct. 26, 2015, 3:36 p.m., B

Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

2015-10-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39670/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Bugs: AURORA-1510

Review Request 39669: Add missing binding for ZooKeeperTestServer when running a local scheduler.

2015-10-26 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39669/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description --

Re: Review Request 39631: Add listener to trigger scheduler shutdown on service failure.

2015-10-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39631/ --- (Updated Oct. 26, 2015, 2:02 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 39629: Remove ChainedPathDetector and root argument from the observer.

2015-10-26 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39629/#review104057 --- Ship it! Ship It! - Maxim Khutornenko On Oct. 25, 2015, 4:33 a.

Re: Review Request 39631: Add listener to trigger scheduler shutdown on service failure.

2015-10-26 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39631/#review104056 --- Ship it! src/main/java/org/apache/aurora/scheduler/SchedulerServi