mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio" <>
Subject Re: Review Request 37336: Added `execute()` method to process::Subprocess
Date Tue, 10 Nov 2015 20:51:24 GMT

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

(Updated Nov. 10, 2015, 8:51 p.m.)

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


Addressed mpark's comments

Summary (updated)

Added `execute()` method to process::Subprocess

Bugs: MESOS-3035

Repository: mesos


The original API for `process::Subprocess` still left a lot of legwork
to do for the caller; we have now added an `execute()` method
that returns a `Future<Subprocess::Result>`.
`Subprocess::Result`, also introduced with this patch, contains useful information
about the command invocation (an `Invocation` struct); 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 (updated)

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

  3rdparty/libprocess/src/subprocess.cpp efe0018d0414c4137fd833c153eb262232e712bc 
  3rdparty/libprocess/src/tests/subprocess_tests.cpp ac600a551fb1a7782ff33cce204b7819497ef54a



make check

(also tested functionality with an anonymous module that exposes an `/execute` endpoint and
runs arbitrary commands, asynchronously,
on an Agent)


Marco Massenzio

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