----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62621/#review186446 -----------------------------------------------------------
@ReviewBot retry - Jordan Ly On Sept. 27, 2017, 5:25 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62621/ > ----------------------------------------------------------- > > (Updated Sept. 27, 2017, 5:25 p.m.) > > > Review request for Aurora, Santhosh Kumar Shanmugham, Stephan Erb, and Bill > Farner. > > > Repository: aurora > > > Description > ------- > > - Allow transitions from any state to STOPPED in CallOrderEnforcingStorage, > including adding a STOPPED -> STOPPED transition so stop() can be used > idempotent. > - Use the StateMachines.checkState method (I wasn't sure if the current > checkInState was designed for anything other than throwing a > TransientStorageException) > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java > 2a5ec9c912979811c4badeee9362c22184d9cbbf > src/main/java/org/apache/aurora/scheduler/storage/Storage.java > 859c96414fbafc1742c77c4bf9bff889d2efa31c > > > Diff: https://reviews.apache.org/r/62621/diff/3/ > > > Testing > ------- > > No unit tests specified. > ./gradlew test > > > Thanks, > > Jordan Ly > >