You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by be...@apache.org on 2013/03/09 02:20:14 UTC

svn commit: r1454636 - /incubator/mesos/branches/0.12.x/src/slave/monitor.cpp

Author: benh
Date: Sat Mar  9 01:20:12 2013
New Revision: 1454636

URL: http://svn.apache.org/r1454636
Log:
Fixed a bug in resource monitor to dispatch to IsolationModule::usage()
instead of invoking usage() directly.

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

Modified:
    incubator/mesos/branches/0.12.x/src/slave/monitor.cpp

Modified: incubator/mesos/branches/0.12.x/src/slave/monitor.cpp
URL: http://svn.apache.org/viewvc/incubator/mesos/branches/0.12.x/src/slave/monitor.cpp?rev=1454636&r1=1454635&r2=1454636&view=diff
==============================================================================
--- incubator/mesos/branches/0.12.x/src/slave/monitor.cpp (original)
+++ incubator/mesos/branches/0.12.x/src/slave/monitor.cpp Sat Mar  9 01:20:12 2013
@@ -179,13 +179,13 @@ void ResourceMonitorProcess::collect(
     return;
   }
 
-  isolation->usage(frameworkId, executorId).onAny(
-      defer(self(),
-            &Self::_collect,
-            lambda::_1,
-            frameworkId,
-            executorId,
-            interval));
+  dispatch(isolation, &IsolationModule::usage, frameworkId, executorId)
+    .onAny(defer(self(),
+                 &Self::_collect,
+                 lambda::_1,
+                 frameworkId,
+                 executorId,
+                 interval));
 }