Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/#review49466 --- Ship it! Ship It! src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java https://reviews.apache.org/r/24206/#comment86534 protected? src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java https://reviews.apache.org/r/24206/#comment86535 missing javadoc - Kevin Sweeney On Aug. 2, 2014, 10:20 a.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/ --- (Updated Aug. 2, 2014, 10:20 a.m.) Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-613 https://issues.apache.org/jira/browse/AURORA-613 Repository: aurora Description --- The JobUpdater will call on an UpdateStrategy to determine how a job update progresses through instances. This will remain a code island until the surrounding features are filled in. Diffs - src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/UpdateStrategy.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategyTest.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategyTest.java PRE-CREATION Diff: https://reviews.apache.org/r/24206/diff/ Testing --- ./gradlew build -Pq Thanks, Bill Farner
Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.
On Aug. 4, 2014, 5:34 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java, line 36 https://reviews.apache.org/r/24206/diff/1/?file=649183#file649183line36 protected? Done. On Aug. 4, 2014, 5:34 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java, line 18 https://reviews.apache.org/r/24206/diff/1/?file=649185#file649185line18 missing javadoc Fixed. - Bill --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/#review49466 --- On Aug. 2, 2014, 5:20 p.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/ --- (Updated Aug. 2, 2014, 5:20 p.m.) Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-613 https://issues.apache.org/jira/browse/AURORA-613 Repository: aurora Description --- The JobUpdater will call on an UpdateStrategy to determine how a job update progresses through instances. This will remain a code island until the surrounding features are filled in. Diffs - src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/UpdateStrategy.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategyTest.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategyTest.java PRE-CREATION Diff: https://reviews.apache.org/r/24206/diff/ Testing --- ./gradlew build -Pq Thanks, Bill Farner
Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/#review49435 --- Ship it! src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java https://reviews.apache.org/r/24206/#comment86502 typo src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java https://reviews.apache.org/r/24206/#comment86501 Missing class comments explaining the meaning? src/test/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategyTest.java https://reviews.apache.org/r/24206/#comment86503 typo - Maxim Khutornenko On Aug. 2, 2014, 5:20 p.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/ --- (Updated Aug. 2, 2014, 5:20 p.m.) Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-613 https://issues.apache.org/jira/browse/AURORA-613 Repository: aurora Description --- The JobUpdater will call on an UpdateStrategy to determine how a job update progresses through instances. This will remain a code island until the surrounding features are filled in. Diffs - src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategy.java PRE-CREATION src/main/java/org/apache/aurora/scheduler/updater/strategy/UpdateStrategy.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/BatchStrategyTest.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/updater/strategy/QueueStrategyTest.java PRE-CREATION Diff: https://reviews.apache.org/r/24206/diff/ Testing --- ./gradlew build -Pq Thanks, Bill Farner