You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Cody Maloney <co...@mesosphere.io> on 2015/04/24 01:32:29 UTC

Re: Review Request 28257: Allow prefix paths in libprocess

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

(Updated April 23, 2015, 11:32 p.m.)


Review request for mesos, Benjamin Hindman, Dominic Hamon, and Joris Van Remoortere.


Changes
-------

Sharing

Changed name to routeWithPrefix.

Added a helper addRoute for common implementation bits between route and routeWithPrefx


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


Repository: mesos


Description
-------

Allow prefix paths in libprocess


Diffs (updated)
-----

  3rdparty/libprocess/include/process/process.hpp 392c74df3e8a122aecd3633dffdeec4bcbd1f097 
  3rdparty/libprocess/src/process.cpp cf4e36489be2c6aa01e838c1c71383f248deab5b 

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


Testing
-------

make distcheck ubuntu 14.04

Manually browse the mesos web UI and verify that things seem to generally work


Thanks,

Cody Maloney