-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45961/
-----------------------------------------------------------
(Updated Aug. 13, 2016, 12:57 a.m.)
Review request for mesos, Benjamin Mahler, Joris Van Remoortere, Michael Park, and Jiang Yan
Xu.
Changes
-------
Addressed review comments and rebased.
Bugs: MESOS-4431
https://issues.apache.org/jira/browse/MESOS-4431
Repository: mesos
Description
-------
o Each shared resource is accouted via its share count. This count is
updated based on the resource operations (such as add and subtract)
in various scenarios such as task launch and terminate at multiple
modules such as master, allocator, sorter, etc.
o Only allow DESTROY if there are no running or pending tasks using
the volume. However, if the volume is in a pending offer to one or
more frameworks, rescind that offer before processing the DESTROY.
o To allow multiple tasks to be launched in the same ACCEPT call
using the same shared resource, we update the allocator and
sorter with additional copies of shared resources to reflect the
true shared count of allocated shared resources.
Diffs (updated)
-----
src/common/resources.cpp 6b7af9179121efbdc5c29484eb042778bcea8288
src/common/resources_utils.cpp 8e881a09500b0966b577e514077460f724868a8d
src/master/allocator/mesos/hierarchical.hpp bdbc6d3b5b959990538f4e3b7b1a3b031d9aea05
src/master/allocator/mesos/hierarchical.cpp 234ef98529964a0b6d3f132426a6c8ccbb1263ee
src/master/allocator/sorter/drf/sorter.hpp bc6bfb2d5d3b32d55be055a0514861b4e7d889bb
src/master/allocator/sorter/drf/sorter.cpp ac85b327fc33d34246788e6a8c8bf5a486c61434
src/master/http.cpp 52dd80b856cf2317c0b73ba54bf501696786088d
src/master/master.hpp 6decff6f4b9c3434de030fd5c06df4c683a7abad
src/master/master.cpp 0bd1a3490a86fede86a3f5f62ce4745b65aae258
src/master/validation.hpp 43d876b00511d4ef652c2685d4950c994b5eef31
src/master/validation.cpp af18e5aef3be59830b0a7b0235bbc739dba1029c
src/tests/master_validation_tests.cpp 9eb82a569f7c48caa96d4d54a93b199ccac74385
src/tests/sorter_tests.cpp 821e30d6574b045d25d4de4f7c3b8ac5346d3002
src/v1/resources.cpp 03ee0cb0bb5abe7fc1ae4cb47c5b6dbcd8d11998
Diff: https://reviews.apache.org/r/45961/diff/
Testing
-------
Tests successful.
Thanks,
Anindya Sinha
|