mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joerg Schad <>
Subject Re: Review Request 45230: Refactored subproces setup functions [1/7].
Date Mon, 28 Mar 2016 16:51:13 GMT

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

(Updated March 28, 2016, 4:51 p.m.)

Review request for mesos and Joris Van Remoortere.

Bugs: MESOS-5049

Repository: mesos


Executing arbitrary setup functions while creating new processes is
dangerous as all functions called have to be async safe. As setup
functions are used for only very few purposes (setsid, chdir, monitoring
and killing a process (see upcoming review) it makes sense to support
them safely via parameters to subprocess. Note this review by itself
-without the following ones- removing the uses of the old interface will
break the build.


  3rdparty/libprocess/include/process/ssl/gtest.hpp 2ca705524c8f9bba3c03eef296dc04a353dd236c

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

  3rdparty/libprocess/src/subprocess.cpp b99bad04f7251169df3bfcec5dee459977440997 



tested entire chain (see


Joerg Schad

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