----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63332/#review189327 -----------------------------------------------------------
src/master/allocator/sorter/drf/sorter.hpp Lines 392 (patched) <https://reviews.apache.org/r/63332/#comment266340> Typo: s/toRemove/oldAllocation/ Could you make sure it compiles and fill out the "Testing Done" section? - Jiang Yan Xu On Oct. 26, 2017, 11:26 a.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63332/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2017, 11:26 a.m.) > > > Review request for mesos, James Peach and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > If calls to these checks fails, log related resources objects which > causes the failure. > > > Diffs > ----- > > src/master/allocator/sorter/drf/sorter.hpp > 77e52dec735d276389643f7f356cd763b2f785e9 > > > Diff: https://reviews.apache.org/r/63332/diff/4/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >