----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68732/#review209003 -----------------------------------------------------------
Ship it! Ship It! - Meng Zhu On Sept. 24, 2018, 5:22 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68732/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2018, 5:22 p.m.) > > > Review request for mesos, Gastón Kleiman and Meng Zhu. > > > Repository: mesos > > > Description > ------- > > This avoids making excessive map lookups each time we calculate the > share for a node in the tree. Now, when the weight is needed, the > value is cached. If the weight gets updated, we update the cached > value. This approach proved cleaner than trying to ensure freshly > constructed nodes have the right weight. > > > Diffs > ----- > > src/master/allocator/sorter/drf/sorter.hpp > 2957a2ea4e6d8fe9cc661be1a42ab3e17a338769 > src/master/allocator/sorter/drf/sorter.cpp > a45f66ff0d77becb6f96f7ff4c288e4ab218e070 > src/master/allocator/sorter/random/sorter.hpp > 825c158322885d52803994c0a827a67aedd609f4 > src/master/allocator/sorter/random/sorter.cpp > fc47756927a6e45183713955e819e0640318f838 > > > Diff: https://reviews.apache.org/r/68732/diff/3/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >