You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ch...@apache.org on 2018/06/12 17:50:37 UTC

mesos git commit: Made `NoopResourceEstimator` return a forever-pending future.

Repository: mesos
Updated Branches:
  refs/heads/master f1c0377a9 -> 8d2cebe5e


Made `NoopResourceEstimator` return a forever-pending future.

There is no need for the noop resource estimator to return a ready
future of empty resource since it is only activated when the user
does not want to use an resource estimator.

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


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

Branch: refs/heads/master
Commit: 8d2cebe5eeb7574caa3d10b7ff6018d8f307fe0e
Parents: f1c0377
Author: Chun-Hung Hsiao <ch...@mesosphere.io>
Authored: Mon Jun 11 16:46:00 2018 -0700
Committer: Chun-Hung Hsiao <ch...@mesosphere.io>
Committed: Tue Jun 12 10:50:02 2018 -0700

----------------------------------------------------------------------
 src/slave/resource_estimators/noop.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/8d2cebe5/src/slave/resource_estimators/noop.cpp
----------------------------------------------------------------------
diff --git a/src/slave/resource_estimators/noop.cpp b/src/slave/resource_estimators/noop.cpp
index 5555c3d..ab05384 100644
--- a/src/slave/resource_estimators/noop.cpp
+++ b/src/slave/resource_estimators/noop.cpp
@@ -37,7 +37,7 @@ public:
 
   Future<Resources> oversubscribable()
   {
-    return Resources();
+    return Future<Resources>();
   }
 };