----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63332/#review189324 -----------------------------------------------------------
Fix it, then Ship it! src/master/allocator/sorter/drf/sorter.hpp Lines 350 (patched) <https://reviews.apache.org/r/63332/#comment266336> Nit: For consistency, I don't think we quote agent IDs (there are no spaces in agent IDs). Here and below? - Jiang Yan Xu On Oct. 26, 2017, 11:12 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:12 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/3/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >
