mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <jos...@mesosphere.io>
Subject Re: Review Request 49177: Added 'relink' semantics to ProcessBase::link.
Date Tue, 28 Jun 2016 23:30:35 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49177/
-----------------------------------------------------------

(Updated June 28, 2016, 4:30 p.m.)


Review request for mesos, Benjamin Hindman, Benjamin Mahler, Gilbert Song, Artem Harutyunyan,
and Jie Yu.


Changes
-------

Rework the enum naming and comments.  And fix the mistaken indenting of the link code (would
have broken things otherwise :)


Bugs: MESOS-5576
    https://issues.apache.org/jira/browse/MESOS-5576


Repository: mesos


Description
-------

The `REMOTE_RELINK` option for `ProcessBase::link` will force the
`SocketManager` to create a new socket if a persistent link already
exists.


Diffs (updated)
-----

  3rdparty/libprocess/include/process/process.hpp 2946e50081c4a418a868083806a26f995c295007

  3rdparty/libprocess/src/process.cpp 9bae71246e751e491be5a989eea8aca29c9aa751 

Diff: https://reviews.apache.org/r/49177/diff/


Testing
-------

make check (OSX)


Thanks,

Joseph Wu


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