You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2018/07/02 22:43:52 UTC
mesos git commit: Removed redundant 'toUnreserved()' call in the
allocator.
Repository: mesos
Updated Branches:
refs/heads/master 2fc1f5b10 -> 47cdd5a7b
Removed redundant 'toUnreserved()' call in the allocator.
This call is no longer necessary as the reservation metadata is now
stripped by `createStrippedScalarQuantity`.
Review: https://reviews.apache.org/r/67773/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/47cdd5a7
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/47cdd5a7
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/47cdd5a7
Branch: refs/heads/master
Commit: 47cdd5a7b6be977638071ec9fc3bc86264a29d1f
Parents: 2fc1f5b
Author: Meng Zhu <mz...@mesosphere.io>
Authored: Mon Jul 2 15:42:33 2018 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Mon Jul 2 15:42:33 2018 -0700
----------------------------------------------------------------------
src/master/allocator/mesos/hierarchical.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/47cdd5a7/src/master/allocator/mesos/hierarchical.cpp
----------------------------------------------------------------------
diff --git a/src/master/allocator/mesos/hierarchical.cpp b/src/master/allocator/mesos/hierarchical.cpp
index cbdfb2b..5a6cd3d 100644
--- a/src/master/allocator/mesos/hierarchical.cpp
+++ b/src/master/allocator/mesos/hierarchical.cpp
@@ -2459,8 +2459,7 @@ bool HierarchicalAllocatorProcess::allocatable(const Resources& resources)
return true;
}
- // We remove the static reservation metadata here via `toUnreserved()`.
- Resources quantity = resources.createStrippedScalarQuantity().toUnreserved();
+ Resources quantity = resources.createStrippedScalarQuantity();
foreach (
const Resources& minResources, CHECK_NOTNONE(minAllocatableResources)) {
if (quantity.contains(minResources)) {