----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24755/ -----------------------------------------------------------
Review request for mesos, Ben Mahler and Jie Yu. Repository: mesos-git Description ------- These functions tend to read and write "everything" and are often used with things like Subprocess. A common pattern is that someone will read from Subprocess::err/out but not hold on to the Subprocess instance which will cause the underlying file descriptors in Subprocess to get closed before the io::read/write has completed. To make this safer, we do what we did with io::redirect and automatically duplicate the file descriptor. This was also a good opportunity to remove io::splice from the public interface since it's not being used and is inherently dangerous. Diffs ----- 3rdparty/libprocess/include/process/io.hpp c77529034311ebb0347309ecaa175d2ef1d95208 3rdparty/libprocess/src/process.cpp d403423e6eec4d1d786ab8799ecb03f4decd7b21 3rdparty/libprocess/src/tests/io_tests.cpp 05ea7bb389f670e190be9942b0782ee4cd2e2c08 3rdparty/libprocess/src/tests/subprocess_tests.cpp 98a4e44bd50cd70c908846b71272525e8b5ce406 Diff: https://reviews.apache.org/r/24755/diff/ Testing ------- make check Thanks, Benjamin Hindman