----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62053/#review184773 -----------------------------------------------------------
Ship it! Modulo deletion of the managed process. - Benjamin Mahler On Sept. 2, 2017, 3:07 p.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62053/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2017, 3:07 p.m.) > > > Review request for mesos, Benjamin Mahler and Jiang Yan Xu. > > > Bugs: MESOS-7921 > https://issues.apache.org/jira/browse/MESOS-7921 > > > Repository: mesos > > > Description > ------- > > The garbage collector had at least two bugs: > > (1) If someone dispatched `manage()` twice in a row the process we're > waiting for will get overwritten which can wreak havoc depending > on when the calls to `link()` happen. > > (2) The garbage collector was deleting after an exited event rather > than actually doing a `wait()`. > > The simpler implementation that this patch introduces is to just > delete the process after doing `ProcessManager::cleanup()`. > > > Diffs > ----- > > 3rdparty/libprocess/include/Makefile.am > c5dc0bb0d2d77987531ead50277940700c62b84f > 3rdparty/libprocess/include/process/gc.hpp > 603bb8bb084a8d2774ab1077da671f659a22a376 > 3rdparty/libprocess/include/process/process.hpp > 8cca782ae89727bc5570afc4ed96c556f14c8c68 > 3rdparty/libprocess/src/process.cpp > afa53537a5c7d4d8b0f4e3b8e04d7d0f2c4c6631 > 3rdparty/libprocess/src/tests/process_tests.cpp > 8d36600701a795a7fa8d73a844657ff98eee6aa7 > > > Diff: https://reviews.apache.org/r/62053/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
