----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45961/ -----------------------------------------------------------
(Updated July 28, 2016, 10:44 p.m.) Review request for mesos, Benjamin Mahler, Joris Van Remoortere, Michael Park, and Jiang Yan Xu. Changes ------- Rebase with master and addressed review comments. Bugs: MESOS-4431 https://issues.apache.org/jira/browse/MESOS-4431 Repository: mesos Description (updated) ------- 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 3dbff24d6859d3b1ed8589cec50170a5202cfbcb 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 1cbec976ccedae81def388c62889624aaf9a794e src/master/master.hpp 6decff6f4b9c3434de030fd5c06df4c683a7abad src/master/master.cpp 060dc7f9730808c7fd9b8f9ecdbde0aac14d135c src/master/validation.hpp 43d876b00511d4ef652c2685d4950c994b5eef31 src/master/validation.cpp 52002beac29c371411348cb026a216e99ac96ab2 src/tests/master_validation_tests.cpp 9eb82a569f7c48caa96d4d54a93b199ccac74385 src/tests/sorter_tests.cpp ade356cbbba5b93a6d3e5c9de30eefd3982d15c1 src/v1/resources.cpp 3c85dc8aa8125962b44e60806ece83a7653d0dc7 Diff: https://reviews.apache.org/r/45961/diff/ Testing ------- Tests successful. Thanks, Anindya Sinha