> On Dec. 18, 2014, 12:08 a.m., Jie Yu wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 678 > > <https://reviews.apache.org/r/29084/diff/2/?file=795197#file795197line678> > > > > Break on 4.4?
Yes that would have broken 4.4.7 again, thanks for catching that! - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29084/#review65384 ----------------------------------------------------------- On Dec. 17, 2014, 11:24 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29084/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2014, 11:24 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 > 1f627386faab61d500d55b65ff6b95bddc800fd5 > src/tests/mesos.hpp bb24222c20cb5458b5c627d2001fc3cb1e542cce > > Diff: https://reviews.apache.org/r/29084/diff/ > > > Testing > ------- > > Added a test. > > > Thanks, > > Ben Mahler > >
