Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review162834 --- Ship it! Master (86d8f2f) is green with this patch.

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-24 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/ --- (Updated Jan. 24, 2017, 8:42 a.m.) Review request for Aurora, David

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161603 --- Master (b75dfb8) is red with this patch.

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161598 --- @ReviewBot retry - Stephan Erb On Jan. 11, 2017, 5:48 p.m.,

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161590 --- Ship it! Ship It! - Zameer Manji On Jan. 11, 2017, 8:48

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161265 --- Master (a94601a) is red with this patch.

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-11 Thread Mehrdad Nurolahzade
> On Jan. 10, 2017, 2:11 p.m., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/preemptor/PreemptorModule.java, > > lines 64-65 > > > > > > We only have a reservation once we have found a match. How

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-11 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/ --- (Updated Jan. 11, 2017, 8:48 a.m.) Review request for Aurora, David

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-10 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161130 --- Ship it! LGTM

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161128 --- Ship it! Master (d4ebb56) is green with this patch.

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-10 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161119 --- @ReviewBot retry - Stephan Erb On Jan. 9, 2017, 10:40 p.m.,

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Mehrdad Nurolahzade
> On Jan. 9, 2017, 1:45 p.m., Aurora ReviewBot wrote: > > Master (d4ebb56) is red with this patch. > > ./build-support/jenkins/build.sh > > > > at > > org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120) > > at > >

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review160974 --- Master (d4ebb56) is red with this patch.

Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/ --- Review request for Aurora, David McLaughlin, Joshua Cohen, Stephan Erb, and