mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio" <ma...@mesosphere.io>
Subject Re: Review Request 37336: Simplified the caller interface to process::Subprocess
Date Fri, 16 Oct 2015 23:03:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37336/
-----------------------------------------------------------

(Updated Oct. 16, 2015, 11:03 p.m.)


Review request for mesos, Joris Van Remoortere and Michael Park.


Bugs: MESOS-3035
    https://issues.apache.org/jira/browse/MESOS-3035


Repository: mesos


Description (updated)
-------

Jira: MESOS-3035

    The original API for `process::Subprocess` still left a lot of legwork
    to do for the caller; we have now added an `executed()` method
    that returns a `Future<CommandResult>` (also introduced with this patch) which
    contains useful information about the command invocation; the exit code;
    stdout and, optionally, stderr too.
    
    Once the Future completes, if successful, the caller will be able to retrieve
    stdout/stderr; whether the command was successful; and whether it received a signal


Diffs
-----

  3rdparty/libprocess/include/process/subprocess.hpp d2341a53aac7c779668ee80983f73158fd44bff5

  3rdparty/libprocess/src/subprocess.cpp d6ea62ed1c914d34e0e189395831c86fff8aac22 
  3rdparty/libprocess/src/tests/subprocess_tests.cpp ab7515325e5db0a4fd222bb982f51243d7b7e39d


Diff: https://reviews.apache.org/r/37336/diff/


Testing
-------

make check


Thanks,

Marco Massenzio


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