mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bernd Mathiske" <>
Subject Re: Review Request 35438: Fixed fetcher cache test race for resource offers when starting tasks and changed corresponding CHECK to EXPECT.
Date Mon, 15 Jun 2015 08:51:56 GMT

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

(Updated June 15, 2015, 1:51 a.m.)

Review request for mesos, Adam B, Benjamin Hindman, Till Toenshoff, and Timothy Chen.


Added TODO regarding the "15 seconds" naked constant when awaiting offers.

Repository: mesos


Follow up to RR, which was not good enough, fixed only
one of two problems.

Using DeclineOffers() instead of Return() should make the master resend offers so we can launch
tasks. See line 205 below.

Following Jie's suggestion, no more CHECK_READY inside launchTask(s)(), but we return a Try
instead and follow call sites with EXPECT_SOME(task(s)).

Diffs (updated)

  src/tests/fetcher_cache_tests.cpp 8bd5dd847fb189d0eeeaa760d3ec8ce3af1c2392 



make check


Bernd Mathiske

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message