----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35699/#review89107 -----------------------------------------------------------
src/master/allocator/mesos/hierarchical.hpp (lines 702 - 703) <https://reviews.apache.org/r/35699/#comment141704> Whoops, Jie just noticed that this isn't correct because 'updatedAllocation' is for the framework only, whereas 'total' and 'available' are for all frameworks on the slave. - Ben Mahler On June 20, 2015, 7:52 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35699/ > ----------------------------------------------------------- > > (Updated June 20, 2015, 7:52 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.hpp > 646ee8c1c0fb824e1d17150b4e96e6281c65358f > > Diff: https://reviews.apache.org/r/35699/diff/ > > > Testing > ------- > > `make check` > > > Thanks, > > Michael Park > >