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

(Updated Aug. 19, 2014, 6:33 p.m.)


Review request for Aurora and Bill Farner.


Changes
-------

Moving "save" into the update controller.


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


Repository: aurora


Description
-------

Implementing pause/resume/abort APIs.

Also, dropping JobUpdater in favor of JobUpdateController to avoid a 
"SchedulerCore fate" (thin proxy interfaces with little to no extra logic).


Diffs (updated)
-----

  src/main/java/org/apache/aurora/scheduler/app/AppModule.java 
61ba4b2c292bb1a76f6133c7212b3cacded6f693 
  src/main/java/org/apache/aurora/scheduler/state/JobUpdater.java 
eb472e422bdc2e324a4aa48e911c7b13c4639674 
  src/main/java/org/apache/aurora/scheduler/state/JobUpdaterImpl.java 
2c6460028f9999c17c6fb04b8b92d91ed53ff333 
  src/main/java/org/apache/aurora/scheduler/state/StateModule.java 
cc1eee4e19c092c0d401558ac01b54627f2d1290 
  src/main/java/org/apache/aurora/scheduler/state/UUIDGenerator.java 
3938c302a68d3ccdda0dc33418a5db00a717bffd 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
0d8d75bfd4db15d5b00302fa37f90a9e85cce95e 
  src/main/java/org/apache/aurora/scheduler/updater/JobUpdateController.java 
5eb51170e159e4b287aba783f3308245ceaa1b02 
  
src/main/java/org/apache/aurora/scheduler/updater/JobUpdateControllerImpl.java 
28fd32ffd18f23b892b7e79b4942bbe3cf4ae7b4 
  src/main/java/org/apache/aurora/scheduler/updater/UpdaterModule.java 
PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/state/JobUpdaterImplTest.java 
90b4e8af94c801792658646a01e5756810b9f787 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 04abeeb78b63d65b6b09cf1ebe277324a18a13b0 
  src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java 
43265fdab1ae900fb828374f6c69e562def2d682 

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


Testing
-------

gradle -Pq build


Thanks,

Maxim Khutornenko

Reply via email to