You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Alexander Rojas <al...@mesosphere.io> on 2015/12/12 17:09:45 UTC
Review Request 41314: Fixed hang introduced in committed version of
HTTP AuthenticatorRouter
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41314/
-----------------------------------------------------------
Review request for mesos, Ben Mahler, Michael Park, and Till Toenshoff.
Repository: mesos
Description
-------
The committed version of the HTTP Authentication remove a cleanup function which
ended causing mesos to hang. This patch reintroduces the discarded cleanup
function and per receiven process message queuing.
Diffs
-----
3rdparty/libprocess/src/process.cpp 43c83e30d5c59bd9845f5126ed79985d7c70b0c2
Diff: https://reviews.apache.org/r/41314/diff/
Testing
-------
At least three times
`make -j8 check`
Additionaly
`./3rdparty/libprocess/libprocess-tests --gtest_filter="HTTPTest.*" --gtest_repeat=10000 --verbose`
`./3rdparty/libprocess/libprocess-tests --gtest_filter="HTTPTest.Endpoints" --gtest_repeat=10000 --verbose`
Thanks,
Alexander Rojas