This is an automated email from the ASF dual-hosted git repository. gilbert pushed a commit to branch 1.7.x in repository https://gitbox.apache.org/repos/asf/mesos.git
commit ac24bca494a02cdf6a3c769418dafd4b107be492 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 8bf8a75..3c5a2e3 100644 --- a/src/launcher/executor.cpp +++ b/src/launcher/executor.cpp @@ -511,15 +511,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; @@ -553,7 +544,7 @@ protected: childHooks); if (s.isError()) { - ABORT("Failed to launch '" + commandString + "': " + s.error()); + ABORT("Failed to launch task subprocess: " + s.error()); } return s->pid();
