> On Aug. 14, 2018, 10:09 p.m., Gastón Kleiman wrote: > > 3rdparty/libprocess/src/tests/process_tests.cpp > > Lines 762-763 (patched) > > <https://reviews.apache.org/r/68327/diff/1/?file=2072112#file2072112line762> > > > > Can we add a description? > > > > Something like: > > > > ``` > > Verifies that its possible to intercept `ExitedEvent` events sent to a > > process upon termination of another process that it is linked to. > > ``
I've updated the test name to FutureExited and DropExited to clarify this, hopefully now the description isn't needed? - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68327/#review207257 ----------------------------------------------------------- On Aug. 13, 2018, 10:22 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68327/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2018, 10:22 p.m.) > > > Review request for mesos, Gastón Kleiman and Meng Zhu. > > > Repository: mesos > > > Description > ------- > > Added tests for FUTURE_EXITED and DROP_EXITED. > > > Diffs > ----- > > 3rdparty/libprocess/src/tests/process_tests.cpp > 17ed2f45c892a3c1c34040a492ad3fdd4a7552c1 > > > Diff: https://reviews.apache.org/r/68327/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >
