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

(Updated Dec. 13, 2015, 5:34 p.m.)


Review request for mesos, Ben Mahler and Joris Van Remoortere.


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


Repository: mesos


Description
-------

The call to `ProcessBase::initialize()` occurs in an asyncrhonous way.
This allows processes to be used before the complete initialization
happens, which is particularly bad for `route` and `install` because
it allows HTTP clients to do request to possible non yet routed or
installed endpoints. This was apparent in the AuthenticationTests.

This patch fixes the race for such tests while a deeper discussion
occurs on the semantics of `ProcessBase::initialize()`


Diffs
-----

  3rdparty/libprocess/src/process.cpp af3cefb59c506863959bf80aedb27ea5d22d7b74 
  3rdparty/libprocess/src/tests/http_tests.cpp 
9fe27039416290296e43c6327c85721342d02cb9 

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


Testing
-------

On OSX, and Ubuntu 14.04:

```bash
$ make -j8 check
$ ./3rdparty/libprocess/libprocess-tests 
--gtest_filter="HttpAuthenticationTest.*" --gtest_repeat=100000 --verbose 
--gtest_break_on_failure
```


Thanks,

Alexander Rojas

Reply via email to