----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12340/#review23586 -----------------------------------------------------------
Ship it! Ship It! - Benjamin Hindman On July 18, 2013, 9:11 p.m., Thomas Marshall wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12340/ > ----------------------------------------------------------- > > (Updated July 18, 2013, 9:11 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Bugs: MESOS-505 > https://issues.apache.org/jira/browse/MESOS-505 > > > Repository: mesos > > > Description > ------- > > As part of the addition of reserved resources to the allocator, we decided to > mark resources with roles. This makes it easy for the allocator to track what > reserved pool different resources came from. This patch updates the current > resource operators to consider roles, so for example only resources with the > same role will be added together. > > > Diffs > ----- > > include/mesos/mesos.proto 233e3c3 > src/common/resources.hpp 42dfb6a > src/common/resources.cpp 8bc1057 > src/tests/resources_tests.cpp 67a11b3 > > Diff: https://reviews.apache.org/r/12340/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Thomas Marshall > >
