aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Review Request 25741: Remove INSTANCE_ADDED and INSTANCE_REMOVED states.
Date Wed, 17 Sep 2014 17:15:19 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for Aurora, David McLaughlin and Maxim Khutornenko.

Repository: aurora


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.


  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/ e09caa63bc0150d7109cb237e80b9efee441dded

  src/test/java/org/apache/aurora/scheduler/storage/log/ ca990e73d80e8456e71a97f0bdd0b6f4530d0135



./gradlew build -Pq


Bill Farner

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