This is an automated email from the ASF dual-hosted git repository. mzhu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mesos.git
commit c186625ea2223ea10d35d16608d10eb03cb385de Author: Andrei Sekretenko <[email protected]> AuthorDate: Wed May 22 16:40:46 2019 +0200 Fixed comment to reflect the existence of sorters other than DRF. Review: https://reviews.apache.org/r/70679/ --- src/master/allocator/mesos/hierarchical.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/master/allocator/mesos/hierarchical.cpp b/src/master/allocator/mesos/hierarchical.cpp index 4fcb3c8..ccca391 100644 --- a/src/master/allocator/mesos/hierarchical.cpp +++ b/src/master/allocator/mesos/hierarchical.cpp @@ -1856,10 +1856,9 @@ void HierarchicalAllocatorProcess::__allocate() break; // Nothing left on this agent. } - // Fetch frameworks according to their fair share. + // Fetch frameworks in the order provided by the sorter. // NOTE: Suppressed frameworks are not included in the sort. CHECK_CONTAINS(frameworkSorters, role); - const Owned<Sorter>& frameworkSorter = frameworkSorters.at(role); foreach (const string& frameworkId_, frameworkSorter->sort()) {
