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 <[email protected]> Authored: Mon Apr 3 14:41:12 2017 +0200 Committer: Alexander Rukletsov <[email protected]> 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()) {
