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();
}