Re: Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34733/#review85438 --- Ship it!

Re: Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34733/ --- (Updated May 27, 2015, 9:22 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34733/#review85441 --- Ship it! Ship It! - Zameer Manji On May 27, 2015, 1:52 p.m.,

Re: Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34733/#review85447 --- Ship it! Master (5724099) is green with this patch.

Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34733/ --- Review request for Aurora, Kevin Sweeney and Zameer Manji. Bugs: AURORA-1339

Re: Review Request 34733: Fixing default TaskReconciler initial delay.

2015-05-27 Thread Maxim Khutornenko
On May 27, 2015, 8:58 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/async/TaskReconciler.java, line 82 https://reviews.apache.org/r/34733/diff/1/?file=973043#file973043line82 Move this validation to the constructor so that it will halt injector creation.

Re: Review Request 34501: Enabling TemporaryStorage to use flagged task store.

2015-05-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34501/#review85449 --- Ship it! Ship It! - Kevin Sweeney On May 22, 2015, 5:31 p.m.,

Re: Review Request 34739: Adding ChainedPathDetector into the GC executor.

2015-05-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34739/#review85479 --- Ship it! Master (5724099) is green with this patch.

Re: Review Request 34484: Daemonize all deadline calls in aurora executor.

2015-05-27 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34484/#review85482 --- Ship it! Ship It! - Joe Smith On May 26, 2015, 1:43 p.m., Brian