You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by al...@apache.org on 2017/04/24 10:14:49 UTC
[3/7] mesos git commit: Improved termination logging in default
executor.
Improved termination logging in default executor.
WSTRINGIFY already includes a preamble in addition to exit status,
no need to repeat it.
Review: https://reviews.apache.org/r/58190
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/674a6190
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/674a6190
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/674a6190
Branch: refs/heads/master
Commit: 674a619052acd5069426f371d8961abbad487648
Parents: 02d8426
Author: Alexander Rukletsov <al...@apache.org>
Authored: Mon Apr 3 14:41:12 2017 +0200
Committer: Alexander Rukletsov <al...@apache.org>
Committed: Mon Apr 24 12:04:51 2017 +0200
----------------------------------------------------------------------
src/launcher/default_executor.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/674a6190/src/launcher/default_executor.cpp
----------------------------------------------------------------------
diff --git a/src/launcher/default_executor.cpp b/src/launcher/default_executor.cpp
index d003c1b..ef60f5b 100644
--- a/src/launcher/default_executor.cpp
+++ b/src/launcher/default_executor.cpp
@@ -802,8 +802,9 @@ protected:
LOG(INFO)
<< "Child container " << container->containerId << " of task '" << taskId
- << "' in state " << stringify(taskState) << " terminated with status "
- << (status.isSome() ? WSTRINGIFY(status.get()) : "unknown");
+ << "' in state " << stringify(taskState) << " "
+ << (status.isSome() ? WSTRINGIFY(status.get())
+ : "terminated with unknown status");
// Shutdown the executor if all the active child containers have terminated.
if (containers.empty()) {