----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51416/#review146826 -----------------------------------------------------------
src/slave/containerizer/mesos/containerizer.cpp (lines 1641 - 1644) <https://reviews.apache.org/r/51416/#comment213549> Is it possible the `reap()` takes forever? - Gilbert Song On Aug. 24, 2016, 10:20 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51416/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2016, 10:20 p.m.) > > > Review request for mesos, Gilbert Song and Kevin Klues. > > > Repository: mesos > > > Description > ------- > > This patch fixed a bug in the destory path. It correctly handles a > race where the container is destroyed while it is being launched. The > idea is to make 'container->status' optional and don't set it until > the container is actually forked. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.hpp > 1f414cfa332d9a3f8b8f04343249e02924e39d89 > src/slave/containerizer/mesos/containerizer.cpp > 8a8985a6627859365c4f87ea0cbb5f89190cd4b3 > > Diff: https://reviews.apache.org/r/51416/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
