mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jay Guo <>
Subject Re: Review Request 50491: Added PIDGroup to libprocess.
Date Mon, 10 Oct 2016 06:43:24 GMT

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

(Updated Oct. 10, 2016, 6:43 a.m.)

Review request for mesos and Joseph Wu.



Bugs: MESOS-5828

Repository: mesos


Log network is not specific to replicated log and the name 'network'
is also confusing. Therefore, PIDGroup is added to libprocess as a
replacement for network. This is a step towards modularizing the
PIDGroup abstraction alongside the `MasterContender` and

Note: The PIDGroup abstraction is a component that discovers all masters
in the group; whereas the `MasterDetector` only discovers the leader
of the group. Both are necessary for the replicated log.

Diffs (updated)

  3rdparty/libprocess/ 020b0e1e4c49805a3c7a61b308d897fc82f5e0ce 
  3rdparty/libprocess/include/ 17c5d11463d0fa16f7039e8395146a62e38deece 
  3rdparty/libprocess/include/process/pid_group.hpp PRE-CREATION 
  3rdparty/libprocess/src/pid_group.cpp PRE-CREATION 
  3rdparty/libprocess/src/tests/pid_group_tests.cpp PRE-CREATION 



see the end of review chain.


Jay Guo

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