Removed misleading note from allocator tests. Review: https://reviews.apache.org/r/42635/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8b5dbd8b Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/8b5dbd8b Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/8b5dbd8b Branch: refs/heads/master Commit: 8b5dbd8b0f7c516c82d7b7fb11b54ee3295af290 Parents: cebbdab Author: Alexander Rukletsov <[email protected]> Authored: Fri Jan 22 00:17:26 2016 -0800 Committer: Benjamin Mahler <[email protected]> Committed: Fri Jan 22 00:17:26 2016 -0800 ---------------------------------------------------------------------- src/tests/hierarchical_allocator_tests.cpp | 6 ------ 1 file changed, 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/8b5dbd8b/src/tests/hierarchical_allocator_tests.cpp ---------------------------------------------------------------------- diff --git a/src/tests/hierarchical_allocator_tests.cpp b/src/tests/hierarchical_allocator_tests.cpp index d11e1c7..f7c82a4 100644 --- a/src/tests/hierarchical_allocator_tests.cpp +++ b/src/tests/hierarchical_allocator_tests.cpp @@ -1569,8 +1569,6 @@ TEST_F(HierarchicalAllocatorTest, RemoveQuota) // Notify allocator of agents, frameworks, quota and current allocations. allocator->setQuota(QUOTA_ROLE, quota1); - // NOTE: We do not report about allocated resources to `framework1` - // here to avoid double accounting. allocator->addFramework( framework1.id(), framework1, @@ -1837,8 +1835,6 @@ TEST_F(HierarchicalAllocatorTest, DRFWithQuota) // Notify allocator of agents, frameworks, quota and current allocations. allocator->setQuota(QUOTA_ROLE, quota1); - // NOTE: We do not report about allocated resources to `framework1` - // here to avoid double accounting. allocator->addFramework( framework1.id(), framework1, @@ -1933,8 +1929,6 @@ TEST_F(HierarchicalAllocatorTest, QuotaAgainstStarvation) SlaveInfo agent2 = createSlaveInfo("cpus:1;mem:512;disk:0"); // Notify allocator of agents, frameworks, and current allocations. - // NOTE: We do not report about allocated resources to `framework1` - // here to avoid double accounting. allocator->addFramework( framework1.id(), framework1,
