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

(Updated Jan. 18, 2016, 3:28 a.m.)

Review request for mesos, Adam B, Alexander Rojas, Benjamin Hindman, Joseph Wu, 
Till Toenshoff, and Timothy Chen.


Addressed Alexander's review. Now conforming with standard pattern of 
"provide()" in "initialize()".

Bugs: MESOS-3235

Repository: mesos


Also inlined the function that awaits fetch contention.

This mutex was prone to causing races at task startup by firmly
blocking an internal libprocess thread. The latch avoids this.

Failing to launch a task due to such a race did not get flagged
by directly related test failures, because the AWAIT catching this
situation was ineffective, having been placed inside a call from the
test. Only the subsequent wait for task completion triggered a test
failure then. By then it was obscured what exactly had happened.

Diffs (updated)

  src/tests/fetcher_cache_tests.cpp 1fb1e213d3c35479789688d1a3a49a3c6058b198 

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


make check
bin/mesos-tests.sh --gtest_repeat=1000 --gtest_break_on_failure 


Bernd Mathiske

Reply via email to