aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <wfar...@apache.org>
Subject Review Request 25300: Add a state machine to react to job update status changes.
Date Wed, 03 Sep 2014 16:02:21 GMT

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

Review request for Aurora, Joshua Cohen and Maxim Khutornenko.


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


Repository: aurora


Description
-------

This will be used by the job updater to gate insertion of JobUpdateEvents.

I chose not to use StateMachine here as it actually added complexity - in this case we don't
benefit from transition callbacks, and the result is based only on the target state (rather
than the transition).


Diffs
-----

  src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java PRE-CREATION

  src/test/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachineTest.java PRE-CREATION


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


Testing
-------

`./gradlew build -Pq`


Thanks,

Bill Farner


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