----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32371/ -----------------------------------------------------------
Review request for Aurora and Maxim Khutornenko. Repository: aurora Description ------- I found myself in here while doing some exploratory changes to `TaskStore`. I started by using `IAssignedTask` and `ITaskConfig` where possible, rather than `IScheduledTask`, and that snowballed into a refactor to extract common functions and predicates. Feel free to push back if you feel this reduces readability or seems like code shuffling with little gain. One upside to this outcome is the imminent post-JDK8 lambda refactor will cause this diff to be a net reduction in code. Diffs ----- src/main/java/org/apache/aurora/scheduler/quota/QuotaManager.java 39e930c4bfe716765908c6d78b58c831b6fb341b src/main/java/org/apache/aurora/scheduler/updater/UpdateFactory.java b53086169aa53d27a39a01cadf8d3c4a8ecb68de src/main/java/org/apache/aurora/scheduler/updater/Updates.java 776278cb64c7ce0419a692143e458801e3b1c37f Diff: https://reviews.apache.org/r/32371/diff/ Testing ------- Thanks, Bill Farner
