> On July 7, 2017, 5:54 a.m., Jie Yu wrote: > > src/master/allocator/mesos/hierarchical.cpp > > Line 652 (original), 652 (patched) > > <https://reviews.apache.org/r/60638/diff/2/?file=1770519#file1770519line652> > > > > Let's update `updateSlaveTotal` so that if the old total is the same as > > the new total, we don't need to update the sorter. That means > > updateSlaveTotal should probably return a bool to indicate if there is any > > change. > > > > This is a great cleanup! Much cleaner now!
Cleaned up. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60638/#review179845 ----------------------------------------------------------- On July 7, 2017, 10:54 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60638/ > ----------------------------------------------------------- > > (Updated July 7, 2017, 10:54 a.m.) > > > Review request for mesos, Benjamin Mahler, Jie Yu, and Jan Schlicht. > > > Bugs: MESOS-7755 > https://issues.apache.org/jira/browse/MESOS-7755 > > > Repository: mesos > > > Description > ------- > > We change the semantics of the 'updateSlave' method present in the > allocator interface. While previously the passed optional resource > argument was interpreted as the amount of (new) oversubscribed > resources, it now represents the new amount of total resources on the > given agent. > > We addtionally add an optimization of > 'HierarchicalAllocatorProcess::updateSlaveTotal' for cases where the > passed total is identical to the current total. This operation is a > no-op now and we prevent updating the sorters. > > > Diffs > ----- > > include/mesos/allocator/allocator.hpp > bec9e0b8ef8d51ec22f9e8af69407ed5ecaa3e8f > src/master/allocator/mesos/allocator.hpp > 2e780c92d5c5132abff32f1ce051c3bab2947f37 > src/master/allocator/mesos/hierarchical.hpp > 5c58cf401de9205e54300e0ce8433995cdf5cb7a > src/master/allocator/mesos/hierarchical.cpp > eb01d8e6b1108866ebc049f9f4a46157823a3541 > src/master/master.cpp 56b170ed70722eba30d98f6e648b6a31580d6b56 > src/tests/hierarchical_allocator_tests.cpp > 2a312a9af4bae679a0a4e7bf45a3c013513c5da2 > > > Diff: https://reviews.apache.org/r/60638/diff/3/ > > > Testing > ------- > > Tested with https://reviews.apache.org/r/60639/. > > > Thanks, > > Benjamin Bannier > >
