----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64954/#review195054 -----------------------------------------------------------
Ship it! Thanks for the benchmarks! LGTM src/test/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImplTest.java Lines 369-372 (original), 356-358 (patched) <https://reviews.apache.org/r/64954/#comment274180> nit: can you just expect 0 - Jordan Ly On Jan. 9, 2018, 6:32 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64954/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2018, 6:32 p.m.) > > > Review request for Aurora, Daniel Knightly and Jordan Ly. > > > Repository: aurora > > > Description > ------- > > This patch sets the stage for performing the bulk of scheduling work in a > separate call path, without holding the write lock. > > Also included is a mechanical refactor pushing the `revocable` flag into > `ResourceRequest` (which was ~always needed as a sibling parameter). > > > Diffs > ----- > > > commons/src/main/java/org/apache/aurora/common/testing/easymock/EasyMockTest.java > c5500b3658437d0a940f953957c880559f5f1b61 > src/jmh/java/org/apache/aurora/benchmark/fakes/FakeOfferManager.java > f0dacd4e90bc142df8b6489ffb33d7265ec03706 > src/main/java/org/apache/aurora/scheduler/TierManager.java > c6ad2b1c48673ca2c14ddd308684d81ce536beca > src/main/java/org/apache/aurora/scheduler/base/TaskTestUtil.java > 22d5a642687c1ddc91071c27abda7d61726b28b9 > > src/main/java/org/apache/aurora/scheduler/configuration/executor/ExecutorSettings.java > 5c987fd051728486172c8afd34219e86d56f00d5 > src/main/java/org/apache/aurora/scheduler/filter/SchedulingFilter.java > bd415909ac1abdb670fde57ead0e391de3e5238b > src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java > cb288bb12d577330888c6a66ddd2baaff4b71e8e > src/main/java/org/apache/aurora/scheduler/offers/HostOffers.java > a01c0a87856a63b1734b4ea336bbbf5fa74b8636 > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java > e90de3eec43a3df9ff019830e927ecff195d1747 > src/main/java/org/apache/aurora/scheduler/offers/OfferManagerImpl.java > 8e806b705d8bee4381ed8afd792d777420405195 > src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictim.java > 69b686639de1eb7d896096e2fadb357fe9ef3141 > > src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilter.java > cf6d348d645d78ab77fcd305307df8a94fea114c > src/main/java/org/apache/aurora/scheduler/resources/ResourceManager.java > d093753b715ffc9ff37448f0266903d6778af024 > src/main/java/org/apache/aurora/scheduler/scheduling/TaskAssigner.java > 87619b55dae0c49029d4f804a52fdcbf9f16a50a > src/main/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImpl.java > 54bd1770274dcedecacccba697aee65b8dd48347 > src/main/java/org/apache/aurora/scheduler/scheduling/TaskScheduler.java > 3c38f953ed830a2e3627705aa6493e9b37dae155 > src/main/java/org/apache/aurora/scheduler/scheduling/TaskSchedulerImpl.java > cff4ab196451fc846827cfc1f48897e645ac3d5d > src/main/java/org/apache/aurora/scheduler/storage/TaskStore.java > 1219215d610e0dbe1f4e78bb8ea57504de0fa329 > > src/main/java/org/apache/aurora/scheduler/storage/durability/WriteRecorder.java > dea8e690d8325596c70f31212029ddcfdf8e4451 > src/main/java/org/apache/aurora/scheduler/updater/UpdateAgentReserver.java > b6909a692b7ea590768ecead08b59277b5577866 > src/test/java/org/apache/aurora/scheduler/TierManagerTest.java > 82e40d509d84c37a19b6a9ef942283d908833840 > > src/test/java/org/apache/aurora/scheduler/events/NotifyingSchedulingFilterTest.java > 64d7a44eaeb627daf7b7bee75ac0be19594a9b25 > > src/test/java/org/apache/aurora/scheduler/filter/SchedulingFilterImplTest.java > 0de90d70afa1bfa00c93875a325b8a3a7ed7b03d > > src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java > c27a662a2a03ee15098ff66229f86da831177a52 > src/test/java/org/apache/aurora/scheduler/offers/OfferManagerImplTest.java > 1e9532bef39591fcc61a2b41a04f2290960d8906 > > src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java > 348386e938ce910c73f1e2e6e2b39f3654a85e7e > > src/test/java/org/apache/aurora/scheduler/scheduling/FirstFitOfferSelectorTest.java > 5e2fdcb646a3362043f171bfc201b83d76524d78 > > src/test/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImplTest.java > 78e12691be0bf20e868ad202aaa306ef29bf5971 > > src/test/java/org/apache/aurora/scheduler/scheduling/TaskSchedulerImplTest.java > ecf2987bf9fa124a4de3a93ffb20b4a807d79b3e > > src/test/java/org/apache/aurora/scheduler/updater/NullAgentReserverTest.java > d8563b86747a7887be9262a621fd761fea832b8c > > src/test/java/org/apache/aurora/scheduler/updater/UpdateAgentReserverImplTest.java > 7f17be0427e3baf12aaa1d176414f7009baeea94 > > > Diff: https://reviews.apache.org/r/64954/diff/2/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >