-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16232/
-----------------------------------------------------------
(Updated Dec. 17, 2013, 3:19 p.m.)
Review request for Aurora, Kevin Sweeney, Maxim Khutornenko, and Bill Farner.
Changes
-------
Improve tests to not use preempt()
Bugs: AURORA-11
https://issues.apache.org/jira/browse/AURORA-11
Repository: aurora
Description
-------
This patch adds a reservation system the preemption flow.
The reservation associates a slave id with a task id for a fixed duration. If
the task attempts to schedule itself during that time period and an offer is
available from that slave then it will be scheduled. If another task attempts
to schedule itself then it will not use the reserved offer.
Diffs (updated)
-----
src/main/java/com/twitter/aurora/scheduler/async/AsyncModule.java
db07841543e554e269f6fe7b36d7f7232af21140
src/main/java/com/twitter/aurora/scheduler/async/Preemptor.java
e5aeb8321e22c51eb3a5dad3d3dd1e26b7121b7d
src/main/java/com/twitter/aurora/scheduler/async/TaskGroups.java
f95f719c5a444b0f8faa4330852e251dd5de528e
src/main/java/com/twitter/aurora/scheduler/async/TaskScheduler.java
fbd82ff70235294cfd27c242f141a585d6bb2396
src/test/java/com/twitter/aurora/scheduler/async/TaskSchedulerImplTest.java
PRE-CREATION
src/test/java/com/twitter/aurora/scheduler/async/TaskSchedulerTest.java
a747f2b1ecbad7263931aeec3b12711096d2469d
src/test/java/com/twitter/aurora/scheduler/state/PubsubTestUtil.java
f9d7fb64728008d0ea6eb424283b58e956e1d50a
Diff: https://reviews.apache.org/r/16232/diff/
Testing
-------
./gradlew clean build
Thanks,
Zameer Manji
