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

(Updated May 19, 2014, 11:25 a.m.)


Review request for Aurora, David McLaughlin and Bill Farner.


Changes
-------

Reworked for compatibility with the new cron code.


Bugs: aurora-417
    https://issues.apache.org/jira/browse/aurora-417


Repository: aurora


Description
-------

Add cron schedule and deschedule calls to the scheduler API.


Diffs (updated)
-----

  src/main/java/org/apache/aurora/scheduler/state/SchedulerCore.java 
226b71ce7749492abd3e1d673382668c9011a1c8 
  src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java 
d377974da4f7efa7f47e06702654bc786299e01c 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
4386a86c0aebdb9666c0232672f8c1bab7458f47 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
66292dc0369941fc62719d49209edc07adc81a53 
  
src/test/java/org/apache/aurora/scheduler/state/BaseSchedulerCoreImplTest.java 
0e17f49e68953b5f043301dbb1b46b96968e1247 
  src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java 
933a56bf3f7165fa84aedcc8d1392e32824fd487 
  src/test/python/apache/aurora/client/api/test_scheduler_client.py 
399d4c614dda93ae0e7d8f38a02379bc0f42e4af 

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


Testing
-------

Ran all tests with gradlew; no errors. (Includes two new tests, that ensure 
that calling the new APIs causes the right internal cron methods to be called.)


Thanks,

Mark Chu-Carroll

Reply via email to