----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60638/#review179845 -----------------------------------------------------------
Fix it, then Ship it! src/master/allocator/mesos/hierarchical.cpp Line 652 (original), 652 (patched) <https://reviews.apache.org/r/60638/#comment254742> 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! - Jie Yu On July 6, 2017, 4:35 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60638/ > ----------------------------------------------------------- > > (Updated July 6, 2017, 4:35 p.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. > > > 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 95e9691b3e3ef101e3e7ddc7a498d5c5bf2276e3 > src/tests/hierarchical_allocator_tests.cpp > 2a312a9af4bae679a0a4e7bf45a3c013513c5da2 > > > Diff: https://reviews.apache.org/r/60638/diff/2/ > > > Testing > ------- > > Tested with https://reviews.apache.org/r/60639/. > > > Thanks, > > Benjamin Bannier > >
