----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64001/#review191877 -----------------------------------------------------------
Ship it! Ship It! - Jan Schlicht On Nov. 24, 2017, 3:07 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64001/ > ----------------------------------------------------------- > > (Updated Nov. 24, 2017, 3:07 p.m.) > > > Review request for mesos and Jie Yu. > > > Repository: mesos > > > Description > ------- > > The allocator method 'updateSlave' can be used to modify either an > agent's capabilities, its total resources, or both. It is expected > that this method return 'true' whenever a change occurred. > > This patch fixes an issue where if the method as implemented in the > hierarchical allocator was called with new capabilities but the > current total resources, 'false' would have been returned, even though > the capabilities where updated. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > cfeeb3bfa3ad7d78ff6e22cfc4adb1f0efa05629 > > > Diff: https://reviews.apache.org/r/64001/diff/2/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >