aurora-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zameer Manji" <zma...@twopensource.com>
Subject Re: Review Request 16232: Add offer reservations to preemption flow
Date Tue, 17 Dec 2013 23:19:45 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message