-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45961/
-----------------------------------------------------------
(Updated July 17, 2016, 6:28 p.m.)
Review request for mesos, Benjamin Mahler, Joris Van Remoortere, Michael Park,
and Jiang Yan Xu.
Changes
-------
Updates based on review comments. Also, fixed filtering based on shared
resources by maintining n copies of shared resources (allocated but not
recovered) in the allocator (for slaves) as well as the various sorters (for
respective clients).
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, resind that offer before processing the DESTROY.
Diffs (updated)
-----
src/common/resources.cpp f6ff92b591c15bc8e93fd85e1896349c3a7bb968
src/common/resources_utils.cpp 8e881a09500b0966b577e514077460f724868a8d
src/master/allocator/mesos/hierarchical.hpp
b72ba16277a3210e4d309b616d185a10e2029a66
src/master/allocator/mesos/hierarchical.cpp
7d4064535a20b93950f5a95eef1ad3f0d37d305b
src/master/allocator/sorter/drf/sorter.hpp
bc6bfb2d5d3b32d55be055a0514861b4e7d889bb
src/master/allocator/sorter/drf/sorter.cpp
ac85b327fc33d34246788e6a8c8bf5a486c61434
src/master/http.cpp d1fe22bd2d9eb6bd82e32cf01720bbe3d8be26d5
src/master/master.hpp ac998b1f5b305a9bff9d9e5cd205a6c3481f9b38
src/master/master.cpp 61eaa4a92741a2d1e3f6624c555a40f6b9240d90
src/master/quota_handler.cpp bf6a613a7bb3c62fd77d1ffde3170749d6c21fa2
src/master/validation.hpp 43d876b00511d4ef652c2685d4950c994b5eef31
src/master/validation.cpp 50ba372d1301d16d1738a3e4f4882b51f9ce06cd
src/master/weights_handler.cpp 9a901e71ba2fbf2ca1c02f658a72d44cfaa5ec62
src/tests/master_validation_tests.cpp
9eb82a569f7c48caa96d4d54a93b199ccac74385
src/tests/sorter_tests.cpp b0e5ef8a55bbcca3553a221bd5691a9c801a04f7
src/v1/resources.cpp 8c3f2d1c1529915a59d47fe37bb3fc7a3267079a
Diff: https://reviews.apache.org/r/45961/diff/
Testing
-------
Tests successful.
Thanks,
Anindya Sinha
