You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2015/09/09 02:06:13 UTC

mesos git commit: Removed a comment from updateAllocation.

Repository: mesos
Updated Branches:
  refs/heads/master b60333b1d -> a1c081746


Removed a comment from updateAllocation.

The allocation for updateAllocation will always be updated after
reserve, unreserve, create and destroy, so there is no need to add the
TODO comments of "Do not log if there is no update".

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


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

Branch: refs/heads/master
Commit: a1c0817463ae272c5cb8d2dcb817e57ee49a8f32
Parents: b60333b
Author: Guangya Liu <gy...@gmail.com>
Authored: Tue Sep 8 17:05:54 2015 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Tue Sep 8 17:05:55 2015 -0700

----------------------------------------------------------------------
 src/master/allocator/mesos/hierarchical.hpp | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/a1c08174/src/master/allocator/mesos/hierarchical.hpp
----------------------------------------------------------------------
diff --git a/src/master/allocator/mesos/hierarchical.hpp b/src/master/allocator/mesos/hierarchical.hpp
index cb4020d..fbf353d 100644
--- a/src/master/allocator/mesos/hierarchical.hpp
+++ b/src/master/allocator/mesos/hierarchical.hpp
@@ -722,7 +722,6 @@ HierarchicalAllocatorProcess<RoleSorter, FrameworkSorter>::updateAllocation(
 
   slaves[slaveId].total = updatedTotal.get();
 
-  // TODO(jieyu): Do not log if there is no update.
   LOG(INFO) << "Updated allocation of framework " << frameworkId
             << " on slave " << slaveId
             << " from " << frameworkAllocation