----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59541/#review176115 -----------------------------------------------------------
Ship it! LGTM! - Anand Mazumdar On May 25, 2017, 6:36 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59541/ > ----------------------------------------------------------- > > (Updated May 25, 2017, 6:36 p.m.) > > > Review request for mesos and Anand Mazumdar. > > > Bugs: MESOS-7552 > https://issues.apache.org/jira/browse/MESOS-7552 > > > Repository: mesos > > > Description > ------- > > This test did not pause the clock; hence, batch allocations could occur > at unpredictable times and cause the test to fail. > > Fix this by pausing the clock and explicitly advancing it as needed to > trigger batch allocations or other events. > > > Diffs > ----- > > src/tests/master_allocator_tests.cpp > 750e030d388d5469399bc79fa723095e62b8f2c5 > > > Diff: https://reviews.apache.org/r/59541/diff/2/ > > > Testing > ------- > > `./src/mesos-tests --gtest_filter="MasterAllocatorTest/0.FrameworkExited" > --gtest_break_on_failure --gtest_repeat=5000` # Linux > > Without this RR, the test fails within a few hundred iterations. > > > Thanks, > > Neil Conway > >
