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 2015/11/20 20:26:18 UTC
Review Request 40410: Libprocess Reinitialization: Move MetricsProcess
instantiation into process::initialize.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40410/
-----------------------------------------------------------
Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.
Bugs: MESOS-3934
https://issues.apache.org/jira/browse/MESOS-3934
Repository: mesos
Description
-------
This must be unified with `process::initialize` so that it also falls under the scope of reinitialization.
Diffs
-----
3rdparty/libprocess/include/process/metrics/metrics.hpp f2e84d8e62df58812b660c858eb3b07777366db4
3rdparty/libprocess/src/metrics/metrics.cpp 0cf844c2f393135236e978862acee88b24581c5c
3rdparty/libprocess/src/process.cpp 8fa0594b671969c2713d1b361f2dbbb07d25b3a8
3rdparty/libprocess/src/tests/system_tests.cpp c5436f72a70f0a55bfe8ba282b5190873a2d9076
Diff: https://reviews.apache.org/r/40410/diff/
Testing
-------
Tests done in a subsequent review.
Thanks,
Joseph Wu
Re: Review Request 40410: Libprocess Reinit: Move MetricsProcess
instantiation into process.cpp.
Posted by Joseph Wu <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40410/
-----------------------------------------------------------
(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 MetricsProcess instantiation into process.cpp.
Bugs: MESOS-3934
https://issues.apache.org/jira/browse/MESOS-3934
Repository: mesos
Description (updated)
-------
The metrics 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/metrics/metrics.hpp 7abfadf6d030c52b7e03e773b67c74db6ad5b5df
3rdparty/libprocess/src/metrics/metrics.cpp 7677dffed98153bc6276aad492a6815e67740ea3
3rdparty/libprocess/src/process.cpp afeddec20495bb9621c3e26b0d425c9419654739
3rdparty/libprocess/src/tests/system_tests.cpp 0f4d0424689522337806ba2227ec4330c700e17e
Diff: https://reviews.apache.org/r/40410/diff/
Testing
-------
Tests done in a subsequent review.
Thanks,
Joseph Wu