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

(Updated Jan. 11, 2016, 12:21 p.m.)

Review request for mesos, Alexander Rojas, Jie Yu, Joseph Wu, Neil Conway, and 
Till Toenshoff.


Separated second line from first line in commit comment.

Summary (updated)

Replaced mutex in HTTP server for fetcher cache tests with latch.

Bugs: MESOS-3235

Repository: mesos

Description (updated)

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.


  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