> On Feb. 23, 2018, 6:06 p.m., Gilbert Song wrote: > > src/tests/containerizer/docker_containerizer_tests.cpp > > Lines 5148 (patched) > > <https://reviews.apache.org/r/65751/diff/3/?file=1963501#file1963501line5148> > > > > Should we use `double`?
The `sleep` command only accepts integer arguments. I opted for `sleep` due to portability, but if there is a highly-portable option which allows sleeping for fractions of a second, that would be great. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65751/#review198192 ----------------------------------------------------------- On Feb. 22, 2018, 11:28 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65751/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2018, 11:28 p.m.) > > > Review request for mesos, Andrei Budnik, Gilbert Song, and Michael Park. > > > Bugs: MESOS-8591 > https://issues.apache.org/jira/browse/MESOS-8591 > > > Repository: mesos > > > Description > ------- > > The new `HungDockerTest` class allows test authors to force > certain Docker daemon calls to be delayed for a specified > duration. > > > Diffs > ----- > > src/tests/containerizer/docker_containerizer_tests.cpp > d1e657050d623ad0412208b3aa3e3101e3654e99 > > > Diff: https://reviews.apache.org/r/65751/diff/3/ > > > Testing > ------- > > Testing details can be found at the end of this chain. > > > Thanks, > > Greg Mann > >
