> On Feb. 11, 2015, 9:59 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/state/TaskAssigner.java, line 64 > > <https://reviews.apache.org/r/30888/diff/1/?file=861003#file861003line64> > > > > fits on one line > > > > ditto elsewhere in this file
Fixed. > On Feb. 11, 2015, 9:59 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/state/TaskAssigner.java, line 102 > > <https://reviews.apache.org/r/30888/diff/1/?file=861003#file861003line102> > > > > use a constant for this Done. - Maxim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30888/#review72026 ----------------------------------------------------------- On Feb. 11, 2015, 9:38 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30888/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2015, 9:38 p.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-909 > https://issues.apache.org/jira/browse/AURORA-909 > > > Repository: aurora > > > Description > ------- > > Offer filtering for static vetoes. Part 1 of 4: TaskAssigner changes to > return a new result object. > > Original RB: https://reviews.apache.org/r/28617/ > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/async/OfferQueue.java > f66383830140e5eaba436f35ebb5192eee65947a > src/main/java/org/apache/aurora/scheduler/async/TaskScheduler.java > ce47ff152e303fd2116bc3b9e91c0c1a8f76f258 > src/main/java/org/apache/aurora/scheduler/state/TaskAssigner.java > e1c29747c9854cf75bf63f6f085cf40ca68989af > src/test/java/org/apache/aurora/scheduler/async/OfferQueueImplTest.java > 4cf602ad32b972c18eb5a81e9b2f59c67859bdb2 > src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerImplTest.java > 5647349854a5e04de749c4d809684a0066d4da06 > src/test/java/org/apache/aurora/scheduler/async/TaskSchedulerTest.java > 6cc13231560996b144101eba36577f49017aba06 > src/test/java/org/apache/aurora/scheduler/state/TaskAssignerImplTest.java > 411a55a8d85f60bb2703468f2d69b64b2736eee4 > > Diff: https://reviews.apache.org/r/30888/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Maxim Khutornenko > >