----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66412/#review200407 -----------------------------------------------------------
src/tests/slave_tests.cpp Lines 9093-9094 (patched) <https://reviews.apache.org/r/66412/#comment281176> Why this change? - Greg Mann On April 3, 2018, 10:10 p.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66412/ > ----------------------------------------------------------- > > (Updated April 3, 2018, 10:10 p.m.) > > > Review request for mesos and Greg Mann. > > > Repository: mesos > > > Description > ------- > > In test `KillTaskGroupBetweenRunTaskParts`, added extra > synchronization to ensure `unmocked__run` finishes before > the test tear down. > > Also updated all dispatch calls to capture by value to avoid > potential races where the arguments might be destroyed > while the dispatch call is still running. > > > Diffs > ----- > > src/tests/slave_tests.cpp 0d7d9726ee39e4c588ea5eb8a54a73fbe1249353 > > > Diff: https://reviews.apache.org/r/66412/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Meng Zhu > >