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 2015/12/10 03:49:17 UTC

[2/2] mesos git commit: Added a reference to MESOS-4111 in the command executor sleep hack.

Added a reference to MESOS-4111 in the command executor sleep hack.


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

Branch: refs/heads/master
Commit: 90771c45b38bddd92e00ecc299d2de818a70eb27
Parents: 7aa7957
Author: Benjamin Mahler <be...@gmail.com>
Authored: Wed Dec 9 18:48:53 2015 -0800
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Wed Dec 9 18:48:53 2015 -0800

----------------------------------------------------------------------
 src/launcher/executor.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/90771c45/src/launcher/executor.cpp
----------------------------------------------------------------------
diff --git a/src/launcher/executor.cpp b/src/launcher/executor.cpp
index f90ea01..09e7de6 100644
--- a/src/launcher/executor.cpp
+++ b/src/launcher/executor.cpp
@@ -524,8 +524,10 @@ private:
 
     driver->sendStatusUpdate(taskStatus);
 
-    // A hack for now ... but we need to wait until the status update
-    // is sent to the slave before we shut ourselves down.
+    // This is a hack to ensure the message is sent to the
+    // slave before we exit the process. Without this, we
+    // may exit before libprocess has sent the data over
+    // the socket. See MESOS-4111.
     os::sleep(Seconds(1));
     driver->stop();
   }