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

(Updated Nov. 4, 2014, 1:19 a.m.)


Review request for mesos, Ben Mahler and Vinod Kone.


Bugs: MESOS-1974
    https://issues.apache.org/jira/browse/MESOS-1974


Repository: mesos-git


Description
-------

The purpose of the refactor is to support persistent disk resources.

Here are the main things I've done in this refactor:
1) Resource objects in Resources are stored in minimal format 
(validated/non-zero). That allows us to kill isAllocatable, allocatable, 
isZero, etc.
2) 'matches' needs to be split into two pieces: one for combining and one for 
removing, in order to support persitent disk resource. For example, one cannot 
combine two Resource object with DiskInfo (it's like two disks), however, you 
can do removal if they are identical.
3) Some of the interfaces are not intuitive (e.g., <=, see details in the 
ticket). I removed them in favor of more explicit interfaces.
4) Unified all the validation code.
5) Adjusted the tests accordingly.


Diffs
-----

  include/mesos/resources.hpp 0e37170 
  src/cli/execute.cpp ddaa20d 
  src/common/resources.cpp e9a0c85 
  src/examples/low_level_scheduler_libprocess.cpp 7ef5ea7 
  src/examples/low_level_scheduler_pthread.cpp 6e233a1 
  src/examples/no_executor_framework.cpp f98a073 
  src/examples/test_framework.cpp 187a611 
  src/master/drf_sorter.cpp 5464900 
  src/master/hierarchical_allocator_process.hpp 31dfb2c 
  src/master/http.cpp a5e34cc 
  src/master/master.cpp 0a5c9a3 
  src/tests/allocator_tests.cpp 58e15aa 
  src/tests/gc_tests.cpp f7747e2 
  src/tests/master_tests.cpp d9dc40c 
  src/tests/mesos.hpp 957e223 
  src/tests/resource_offers_tests.cpp 060039e 
  src/tests/resources_tests.cpp 3e50889 
  src/tests/slave_recovery_tests.cpp 4fb357b 
  src/tests/sorter_tests.cpp 0516ab5 

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


Testing
-------

make check


Thanks,

Jie Yu

Reply via email to