This is an automated email from the ASF dual-hosted git repository. gilbert pushed a commit to branch 1.4.x in repository https://gitbox.apache.org/repos/asf/mesos.git
commit 2f889932469ecd3d8557a44d9c17566a18d61c95 Author: Gilbert Song <[email protected]> AuthorDate: Mon Feb 11 12:51:26 2019 -0800 Cleaned up command executor redundant command string. Review: https://reviews.apache.org/r/69948/ (cherry picked from commit b9970366a988aef6d735766bf770c32f50aa2bfe) --- src/launcher/executor.cpp | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/src/launcher/executor.cpp b/src/launcher/executor.cpp index 81935eb..1984486 100644 --- a/src/launcher/executor.cpp +++ b/src/launcher/executor.cpp @@ -486,15 +486,6 @@ protected: initPath = "/proc/self/fd/" + stringify(memFd.get()); #endif // __linux__ - // TODO(tillt): Consider using a flag allowing / disallowing the - // log output of possibly sensitive data. See MESOS-7292. - string commandString = strings::format( - "%s %s <POSSIBLY-SENSITIVE-DATA>", - initPath, - MesosContainerizerLaunch::NAME).get(); - - LOG(INFO) << "Running '" << commandString << "'"; - // Fork the child using launcher. vector<string> argv(2); argv[0] = MESOS_CONTAINERIZER; @@ -518,7 +509,7 @@ protected: {Subprocess::ChildHook::SETSID()}); if (s.isError()) { - ABORT("Failed to launch '" + commandString + "': " + s.error()); + ABORT("Failed to launch task subprocess: " + s.error()); } return s->pid();
