> On April 8, 2015, 2:09 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.hpp, line 349 > > <https://reviews.apache.org/r/31667/diff/4/?file=888331#file888331line349> > > > > Does it need to be inside the loop?
Absolutely not! Good catch. Fixed. > On April 8, 2015, 2:09 p.m., Alexander Rukletsov wrote: > > src/master/allocator/sorter/drf/sorter.cpp, line 254 > > <https://reviews.apache.org/r/31667/diff/4/?file=888333#file888333line254> > > > > Is it a TODO or a NOTE? Should have been a `NOTE`. Fixed. > On April 8, 2015, 2:09 p.m., Alexander Rukletsov wrote: > > src/master/allocator/sorter/drf/sorter.cpp, line 258 > > <https://reviews.apache.org/r/31667/diff/4/?file=888333#file888333line258> > > > > `_allocations` is per-client allocation, let's reflect this in the name > > and get rid of leading underscore. How about `clientAllocations`? Fixed to `s/_allocations/clientAllocation/`. Dropped the 's' because it's a single allocation for a client out of `allocations`. The semantics are the same as the `allocation` function. Also `s/_resources/totalResources/`. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31667/#review79345 ----------------------------------------------------------- On April 17, 2015, 4:57 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31667/ > ----------------------------------------------------------- > > (Updated April 17, 2015, 4:57 a.m.) > > > Review request for mesos, Alexander Rukletsov and Ben Mahler. > > > Bugs: MESOS-2373 > https://issues.apache.org/jira/browse/MESOS-2373 > > > Repository: mesos > > > Description > ------- > > `Sorter` changes: > > - Augmented `add`, `remove`, `allocated`, `unallocated`, `update` with > `SlaveID`. > - `allocation` returns `hashmap<SlaveID, Resources>`. > > `DRFSorter` changes: > > - `allocations` is updated from `hashmap<std::string, Resources>` to > `hashmap<std::string, hashmap<SlaveID, Resources>>`. > - `resources` is updated from `Resources` to `hashmap<SlaveID, Resources>`. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.hpp > 9f9a631ee52a3ab194e678f9be139e8dabc7f3db > src/master/allocator/sorter/drf/sorter.hpp > 4366710d6530b784aa5094813328d0e338239ba0 > src/master/allocator/sorter/drf/sorter.cpp > 2f69f384b95ff20d3ee429a4570a8cffa74d8e8b > src/master/allocator/sorter/sorter.hpp > e2efb27b11dbea42dd73f81e5db0d6d2b0a6034b > src/tests/sorter_tests.cpp 42442353afe7bd3d1a5b43992f8ae191ac19bdcd > > Diff: https://reviews.apache.org/r/31667/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >
