aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.
Date Tue, 14 Jan 2014 23:27:03 GMT

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

Review request for Aurora, Kevin Sweeney and Maxim Khutornenko.

Repository: aurora


Sorry for the monster diff.  The intertwined nature of StateManagerImpl and TaskStateMachine
made it pretty challenging to do this partially.  I hope the end result is much easier to

The "big picture" for this change is that the closures inside TaskStateMachine no longer drop
items onto a work queue that feeds back into StateManagerImpl.  Instead, it returns these
actions in a TransitionResult.  I intend to improve this further in the future by exposing
only a helper function in TaskStateMachine, to guarantee the one-time-use semantic.


  src/main/java/org/apache/aurora/scheduler/state/ PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/state/ 2bdd4591f2182fe6c44d46f778be562d30eb2392

  src/main/java/org/apache/aurora/scheduler/state/ 2b8ca095d8f108d516a43af8de4ff451ed9a8924

  src/main/java/org/apache/aurora/scheduler/state/ 11d283d31883b865b224d5af169dd5c42875021d

  src/main/java/org/apache/aurora/scheduler/state/ aff74d535eb1237beafbcdf936d5ccc7101377c9

  src/main/java/org/apache/aurora/scheduler/storage/ 79f56052a25ba756208e747dc5d198f30f0c4900

  src/main/java/org/apache/aurora/scheduler/storage/ 7fe297a37aee0cb1be495e6a568b66271ee7bc3d

  src/test/java/org/apache/aurora/scheduler/state/ 53793000de08fe80c0334241d332e3a50fca222a

  src/test/java/org/apache/aurora/scheduler/state/ f44ee589430c2d4c0e014a705fd24b1f2fe08f36



./gradlew build


Bill Farner

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