----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29084/#review65211 -----------------------------------------------------------
src/master/allocator.hpp <https://reviews.apache.org/r/29084/#comment108241> if this is a Shared, why are you passing by reference? this means the reference count won't be updated and the lifetime of the Shared may end before this method returns. Unless I'm missing something. - Dominic Hamon On Dec. 15, 2014, 10:28 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29084/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2014, 10:28 p.m.) > > > Review request for mesos and Jie Yu. > > > Bugs: MESOS-2099 > https://issues.apache.org/jira/browse/MESOS-2099 > > > Repository: mesos-git > > > Description > ------- > > The idea here is that we want the framework to be able to transform the > resources allocated to it, and these Transformations need to be reflected in > the allocator. > > Note that Transformations should not be updating the quantity, or the static > roles of the resources. I've left this as a TODO and hopefully we can enforce > this in the Transformation abstraction itself. > > > Diffs > ----- > > src/master/allocator.hpp f4068aad42a7df38544718dcd171ff239dc1e39a > src/master/hierarchical_allocator_process.hpp > 95fa520a3947474472cd36987c2342e1bfb51cfe > src/tests/hierarchical_allocator_tests.cpp > c43d3520a06dda6b48e74edfd692b3431fb639d2 > src/tests/mesos.hpp bb24222c20cb5458b5c627d2001fc3cb1e542cce > > Diff: https://reviews.apache.org/r/29084/diff/ > > > Testing > ------- > > Added a test. > > > Thanks, > > Ben Mahler > >
