aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Sweeney" <kevi...@apache.org>
Subject Re: Review Request 19767: CronScheduler based on Quartz
Date Wed, 23 Apr 2014 18:40:22 GMT

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

(Updated April 23, 2014, 11:39 a.m.)


Review request for Aurora, Maxim Khutornenko and Bill Farner.


Changes
-------

Rebase to pick up UI removal


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


Repository: aurora


Description
-------

This introduces a new CronScheduler based on Quartz and removes the NoopCronScheduler.


Diffs (updated)
-----

  build.gradle 459cd85f4f239e19e013e74a4745edc7994a9411 
  src/main/java/org/apache/aurora/scheduler/MesosTaskFactory.java 86bbc29c64dd62037ad6bc51b8daa30115eaf74c

  src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java bf3d7a36a575bb9d64f4dd851c63fbebda1e61b8

  src/main/java/org/apache/aurora/scheduler/async/TaskGroups.java 6aff091d9afefc73af666f980ef96acd726aec20

  src/main/java/org/apache/aurora/scheduler/base/JobKeys.java db1bec4f508c8908f212aa541fb86e041a8c471c

  src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java 82034e008e5dbaa3124dc154cdc6c5e9767ca87f

  src/main/java/org/apache/aurora/scheduler/cron/CronJobManager.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/CronPredictor.java df0c37839c5da54795404c18ff9fc93084cd32e4

  src/main/java/org/apache/aurora/scheduler/cron/CronScheduler.java 56e9950fd94ae1e3dbd96baec00b7e6b262fbe34

  src/main/java/org/apache/aurora/scheduler/cron/CrontabEntryTest.java 2bb848a7f5f096b1c85596e4130f0656e9a4401e

  src/main/java/org/apache/aurora/scheduler/cron/SanitizedCronJob.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/noop/NoopCronModule.java e0935f5eab8a101f4ce1831f260f9a23137124ce

  src/main/java/org/apache/aurora/scheduler/cron/noop/NoopCronPredictor.java 7b25152c0258e10be21b801cae1444c518367fa7

  src/main/java/org/apache/aurora/scheduler/cron/noop/NoopCronScheduler.java a31551c77818c17ee0f9f71b5ab458a3b853dc6a

  src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJobFactory.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronJobManagerImpl.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronLifecycle.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronModule.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronPredictorImpl.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronSchedulerImpl.java PRE-CREATION

  src/main/java/org/apache/aurora/scheduler/cron/quartz/Quartz.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/cron/testing/AbstractCronIT.java 61b01d2575d6cfce069e77c39bfa8f8680cf4298

  src/main/java/org/apache/aurora/scheduler/http/Cron.java 80a398a5f297558a25c0a0c63afcb049a9558e44

  src/main/java/org/apache/aurora/scheduler/http/ServletModule.java 983101277ffbd1c4017b29f4c86e61315f1bcc78

  src/main/java/org/apache/aurora/scheduler/http/StructDump.java efea75f3d5a5f4c538c63cc15d5a004d891c2a4a

  src/main/java/org/apache/aurora/scheduler/state/CronJobManager.java 4bd190cd608c167f31f06fba6185ce825926e37e

  src/main/java/org/apache/aurora/scheduler/state/LockManagerImpl.java 5696485e5beb9b7bf4ccee8b6189f25db51aff39

  src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java 536927949c915363cccf074449d089314ecf6996

  src/main/java/org/apache/aurora/scheduler/state/StateModule.java 7d26082b74f62f35865e0343f9ba8b475e075d62

  src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 84151a5308c12b3bee7cf5fd662776e574e8fadf

  src/main/resources/org/apache/aurora/scheduler/cron/testing/cron-schedule-predictions.json
 
  src/test/java/org/apache/aurora/scheduler/cron/ExpectedPrediction.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/cron/noop/NoopCronIT.java a9b85d0983dcfee89101a5e774ba86ee11708c68

  src/test/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJobTest.java PRE-CREATION

  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronIT.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronJobManagerImplTest.java PRE-CREATION

  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronPredictorImplTest.java PRE-CREATION

  src/test/java/org/apache/aurora/scheduler/cron/quartz/QuartzTestUtil.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/state/BaseSchedulerCoreImplTest.java d7dbfaa76b277b028e56ed633ea2a032935e26db

  src/test/java/org/apache/aurora/scheduler/state/CronJobManagerTest.java fa9cb757936542c483699b3fc6bba944d717abac

  src/test/java/org/apache/aurora/scheduler/state/LockManagerImplTest.java c8ad55d8d48f7e96180846ab515dd4df3d8ed79e

  src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java f530c530548444c57d9aeb5ed100a5fda5bd2953

  src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java e212174ed089fdcf28fa679318fe216917a40b99


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


Testing
-------

./gradlew build


File Attachments
----------------

Coverage report.
  https://reviews.apache.org/media/uploaded/files/2014/04/03/6b5f24f5-86a5-43d2-8a0e-d69fd24f7d2a__Screenshot_from_2014-04-02_203443.png


Thanks,

Kevin Sweeney


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