----------------------------------------------------------- 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. Changes ------- Was getting checkstyle exception, needed to propogate the cause of the exception. 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 (updated) ----- 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/2/ Changes: https://reviews.apache.org/r/62621/diff/1-2/ Testing ------- No unit tests specified. ./gradlew test Thanks, Jordan Ly
