You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Joseph Wu <jo...@mesosphere.io> on 2016/04/14 22:28:48 UTC
Re: Review Request 40413: Libprocess Reinit: Move ReaperProcess
instantiation into process.cpp.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40413/
-----------------------------------------------------------
(Updated April 14, 2016, 1:28 p.m.)
Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.
Changes
-------
Reviving this chain...
Summary (updated)
-----------------
Libprocess Reinit: Move ReaperProcess instantiation into process.cpp.
Bugs: MESOS-3934
https://issues.apache.org/jira/browse/MESOS-3934
Repository: mesos
Description (updated)
-------
The reaper singleton must be unified with `process::initialize` so
that it also falls under the scope of reinitialization. The singleton
must also not be guarded by `Once`.
Diffs (updated)
-----
3rdparty/libprocess/include/process/reap.hpp 1a9709c618c5ddc9d2b7492cc1855a11f1fc4fb9
3rdparty/libprocess/src/process.cpp afeddec20495bb9621c3e26b0d425c9419654739
3rdparty/libprocess/src/reap.cpp 110386842d5eeccfbc4e48bfca88d79a9d087fb0
Diff: https://reviews.apache.org/r/40413/diff/
Testing
-------
Tests done in a subsequent review.
Thanks,
Joseph Wu