> On March 25, 2013, 10:12 p.m., Benjamin Hindman wrote: > > src/tests/master_tests.cpp, line 929 > > <https://reviews.apache.org/r/10113/diff/2/?file=274386#file274386line929> > > > > I think we need os::sleep which takes a Duration and calls usleep under > > the covers!
Agreed. Filed https://issues.apache.org/jira/browse/MESOS-415 against me. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10113/#review18369 ----------------------------------------------------------- On March 25, 2013, 7:47 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10113/ > ----------------------------------------------------------- > > (Updated March 25, 2013, 7:47 p.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > See summary. > > > This addresses bug MESOS-412. > https://issues.apache.org/jira/browse/MESOS-412 > > > Diffs > ----- > > src/tests/master_tests.cpp 16f7637c144160448ca8f3eb50924be3a70d2c3d > > Diff: https://reviews.apache.org/r/10113/diff/ > > > Testing > ------- > > sudo GLOG_v=2 ./bin/mesos-tests.sh > --gtest_filter="*ShutdownUnregisteredExecutor*" --verbose --gtest_repeat=-1 > --gtest_break_on_failure > > > Thanks, > > Vinod Kone > >
