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

Review request for Aurora, David McLaughlin and Maxim Khutornenko.


Repository: aurora


Description
-------

Since we're changing fields in `JobUpdateConfiguration` (being renamed to 
`JobUpdateInstructions`) to represent the delta incurred by a job update 
(AURORA-718), i believe we can remove these two states, as addition/removal can 
be inferred from the delta.

Furthermore, this allows the job updater's internal state machine to translate 
directly into the remaining `JobUpdateActions`, and hides details of the 
non-atomic actions necessary to move an instance from state A to state B.


Diffs
-----

  src/main/resources/org/apache/aurora/scheduler/http/ui/js/services.js 
5d6299f9de6eccd0f1332e11d57dfb910d956011 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
3d0beeaed74aafcec0e24725f443e53a67f6c3a0 
  
src/test/java/org/apache/aurora/scheduler/storage/db/DBJobUpdateStoreTest.java 
e09caa63bc0150d7109cb237e80b9efee441dded 
  src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java 
ca990e73d80e8456e71a97f0bdd0b6f4530d0135 

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


Testing
-------

./gradlew build -Pq


Thanks,

Bill Farner

Reply via email to