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/03/14 20:48:09 UTC

mesos git commit: Renamed a variable in allocator to be consistent with others.

Repository: mesos
Updated Branches:
  refs/heads/master 82c50c0e0 -> 70d63d951


Renamed a variable in allocator to be consistent with others.

Renamed `getQuotaRoleAllocatedResources` to
`getQuotaRoleAllocatedScalarQuantities` to be consistent with
other naming.

Review: https://reviews.apache.org/r/66043/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/70d63d95
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/70d63d95
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/70d63d95

Branch: refs/heads/master
Commit: 70d63d95126e912a75fc4003695c7019d82f9e98
Parents: 82c50c0
Author: Meng Zhu <mz...@mesosphere.io>
Authored: Wed Mar 14 13:47:43 2018 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Wed Mar 14 13:47:43 2018 -0700

----------------------------------------------------------------------
 src/master/allocator/mesos/hierarchical.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/70d63d95/src/master/allocator/mesos/hierarchical.cpp
----------------------------------------------------------------------
diff --git a/src/master/allocator/mesos/hierarchical.cpp b/src/master/allocator/mesos/hierarchical.cpp
index 0e8c2c4..32e8895 100644
--- a/src/master/allocator/mesos/hierarchical.cpp
+++ b/src/master/allocator/mesos/hierarchical.cpp
@@ -1565,7 +1565,7 @@ void HierarchicalAllocatorProcess::__allocate()
   // order to meet its quota guarantee.
   //
   // NOTE: Revocable resources are excluded in `quotaRoleSorter`.
-  auto getQuotaRoleAllocatedResources = [this](const string& role) {
+  auto getQuotaRoleAllocatedScalarQuantities = [this](const string& role) {
     CHECK(quotas.contains(role));
 
     // NOTE: `allocationScalarQuantities` omits dynamic reservation,
@@ -1613,7 +1613,7 @@ void HierarchicalAllocatorProcess::__allocate()
 
     // Then add allocated resoruces.
     rolesConsumedQuotaScalarQuantites[role] +=
-      getQuotaRoleAllocatedResources(role);
+      getQuotaRoleAllocatedScalarQuantities(role);
 
     // Lastly subtract allocated reservations on each agent.
     const hashmap<SlaveID, Resources> allocations =