-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16874/
-----------------------------------------------------------

(Updated Jan. 16, 2014, 6:22 p.m.)


Review request for Aurora, Maxim Khutornenko and Bill Farner.


Changes
-------

Rebase onto latest master.


Bugs: AURORA-38
    https://issues.apache.org/jira/browse/AURORA-38


Repository: aurora


Description
-------

Deprecate CronCollisionPolicy.RUN_OVERLAP.

To prevent cluster starvation, the scheduler now ignores new cron jobs
with RUN_OVERLAP and behaves like CANCEL_NEW for existing ones. The
choice of CANCEL_NEW was arbitrary but IMO fits with the mental
preemption model of Aurora better (you're never guaranteed that a new
instance will schedule but Aurora doesn't tend to kill existing
instances outside of an established preemption routine).


Diffs (updated)
-----

  
src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
 f7890186a0fc10ee9fcceff63bbb671dc41eb3be 
  src/main/java/org/apache/aurora/scheduler/state/CronJobManager.java 
b772712b8b3c1e7eb53bfbf55e9d2fc4ee9d5ad9 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
927552a2c9ee90bc282583d99d3cf49a2b1edc25 
  
src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java
 c9ffc70c537ad2b4e72fa6aaaf39ab4478a230af 
  
src/test/java/org/apache/aurora/scheduler/state/BaseSchedulerCoreImplTest.java 
9291fe668a30c32bed6791351c0fa905e57c26c1 
  src/test/java/org/apache/aurora/scheduler/state/CronJobManagerTest.java 
c67912365e1ca40a6d0d57cce6dc169306d17f10 

Diff: https://reviews.apache.org/r/16874/diff/


Testing
-------

./gradlew build


Thanks,

Kevin Sweeney

Reply via email to