You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by an...@apache.org on 2016/08/10 18:28:56 UTC

[4/5] mesos git commit: Removed a no longer valid comment.

Removed a no longer valid comment.

After moving the existing command executor to the unversioned
namespace, we won't be needing support for versioned health
check messages anytime soon. This comment was written when
we were using the `v1` messages for the command executor.

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


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

Branch: refs/heads/master
Commit: 00709d0dbb71d61242b902d3d324fa2dd5f12adc
Parents: 64842e4
Author: Anand Mazumdar <an...@apache.org>
Authored: Wed Aug 10 11:04:13 2016 -0700
Committer: Anand Mazumdar <an...@apache.org>
Committed: Wed Aug 10 11:04:13 2016 -0700

----------------------------------------------------------------------
 src/launcher/executor.cpp | 3 ---
 1 file changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/00709d0d/src/launcher/executor.cpp
----------------------------------------------------------------------
diff --git a/src/launcher/executor.cpp b/src/launcher/executor.cpp
index 0e23d4f..1dfe9a4 100644
--- a/src/launcher/executor.cpp
+++ b/src/launcher/executor.cpp
@@ -237,9 +237,6 @@ public:
 protected:
   virtual void initialize()
   {
-    // TODO(qianzhang): Currently, the `mesos-health-check` binary can only
-    // send unversioned `TaskHealthStatus` messages. This needs to be revisited
-    // as part of MESOS-5103.
     install<TaskHealthStatus>(
         &CommandExecutor::taskHealthUpdated,
         &TaskHealthStatus::task_id,