-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5870/
-----------------------------------------------------------

(Updated Aug. 16, 2012, 9:13 p.m.)


Review request for mesos.


Changes
-------

Updated to latest review. In particular, the isStatic bool has been removed 
from HierarchicalAllocatorProcess, which now assumes that it should do static 
allocations any time a slave has its dedicated_user and/or dedicated_framework 
flag set (which have also been moved from being part of attributes to being top 
level flags and optional members of SlaveInfo).


Description
-------

Simple implementation of a static allocator.

This patch depends on 2 other pending code review:
https://reviews.apache.org/r/5599/
https://reviews.apache.org/r/5913/


This addresses bug MESOS-230.
    https://issues.apache.org/jira/browse/MESOS-230


Diffs (updated)
-----

  include/mesos/mesos.proto 72fccde 
  src/master/hierarchical_allocator_process.hpp PRE-CREATION 
  src/slave/flags.hpp 0c7917f 
  src/slave/slave.cpp 4efd41e 
  src/tests/allocator_tests.cpp 98efb0d 

Diff: https://reviews.apache.org/r/5870/diff/


Testing
-------

make check on Lion
new test in allocator_process_tests.cpp


Thanks,

Thomas Marshall

Reply via email to