mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio" <>
Subject Re: Review Request 36424: Created a command executor helper method.
Date Sat, 18 Jul 2015 02:00:53 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 18, 2015, 2 a.m.)

Review request for mesos, Benjamin Hindman, Ben Mahler, Cody Maloney, and Paul Brett.


Forgotten to convert comments to javadoc format.

Bugs: MESOS-3035

Repository: mesos


Jira: MESOS-2902

While researching how to execute an arbitrary script
to detect the Master IP address, it emerged clearly that
a helper method to execute an arbitrary command/script on
a node and obtain either stdout or stderr would have been
useful and avoided a lot of code repetition.

This could not be ultimately used for the purpose at hand,
but I believe it to be useful enough (particularly, to avoid
people doing "coding by copy&paste" and/or waste time
researching the same functionality).

This would also be beneficial in MESOS-2830 and MESOS-2834
factoring out the remote command execution logic.

Diffs (updated)

  3rdparty/libprocess/include/process/subprocess.hpp 310cb4f8e4e2faa5545dffd196d7490c868bc5d6

  3rdparty/libprocess/src/tests/subprocess_tests.cpp f6acb204582a9e696c3b09d4e4c543bb052e97d4



make check


Marco Massenzio

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message