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 2014/12/19 03:53:42 UTC

mesos git commit: Removed an invalid CHECK in the allocator.

Repository: mesos
Updated Branches:
  refs/heads/master cd7019cd9 -> 1d5380e4a


Removed an invalid CHECK in the allocator.


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

Branch: refs/heads/master
Commit: 1d5380e4aaf3972819a6a640c31d4766dbf9590d
Parents: cd7019c
Author: Benjamin Mahler <be...@gmail.com>
Authored: Thu Dec 18 18:53:27 2014 -0800
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Thu Dec 18 18:53:27 2014 -0800

----------------------------------------------------------------------
 src/master/hierarchical_allocator_process.hpp | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/1d5380e4/src/master/hierarchical_allocator_process.hpp
----------------------------------------------------------------------
diff --git a/src/master/hierarchical_allocator_process.hpp b/src/master/hierarchical_allocator_process.hpp
index 6e2e958..df55a57 100644
--- a/src/master/hierarchical_allocator_process.hpp
+++ b/src/master/hierarchical_allocator_process.hpp
@@ -571,10 +571,6 @@ HierarchicalAllocatorProcess<RoleSorter, FrameworkSorter>::transformAllocation(
 
   Resources allocation = frameworkSorter->allocation(frameworkId.value());
 
-  // The role sorter should only contain the unreserved allocation.
-  CHECK_EQ(allocation.unreserved(),
-           roleSorter->allocation(frameworks[frameworkId].role));
-
   // Update the allocated resources.
   // TODO(bmahler): Check transformation invariants! Namely,
   // we don't want the quantity or the static roles of the