> On March 12, 2014, 4:07 p.m., Dominic Hamon wrote: > > src/master/drf_sorter.cpp, line 106 > > <https://reviews.apache.org/r/19090/diff/1/?file=517048#file517048line106> > > > > client.allocations = it->allocations + 1;
I wanted this to be clearly two steps: (1) copy 'Client' and (2) increment 'allocations'. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19090/#review36923 ----------------------------------------------------------- On March 12, 2014, 5:04 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19090/ > ----------------------------------------------------------- > > (Updated March 12, 2014, 5:04 a.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Bugs: MESOS-1086 > https://issues.apache.org/jira/browse/MESOS-1086 > > > Repository: mesos-git > > > Description > ------- > > See bug for more description. > > > Diffs > ----- > > src/master/drf_sorter.hpp f3fab71424fa10f87c87b733cc678fd90579a814 > src/master/drf_sorter.cpp bbf8270603e26a1b63e0f96bd5aa9c4ee4eb6d25 > > Diff: https://reviews.apache.org/r/19090/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
