> On 六月 14, 2018, 11:18 p.m., Gilbert Song wrote: > > src/slave/containerizer/docker.cpp > > Lines 1383-1385 (patched) > > <https://reviews.apache.org/r/67585/diff/2/?file=2040459#file2040459line1383> > > > > Please follow this format: > > > > ``` > > if (containers_[containerId]->state == Container::DESTROYING) { > > return Failure( > > "Container is being destroyed during launching excutor > > container"); > > } > > ```
ok - CHEN ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67585/#review204804 ----------------------------------------------------------- On 六月 15, 2018, 5:21 p.m., CHEN RUNCONG wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67585/ > ----------------------------------------------------------- > > (Updated 六月 15, 2018, 5:21 p.m.) > > > Review request for mesos and Gilbert Song. > > > Bugs: MESOS-6092 > https://issues.apache.org/jira/browse/MESOS-6092 > > > Repository: mesos > > > Description > ------- > > Fixed the issue:if the container is destroyed in the mean time, then we will > potentially segfault here. > > > Diffs > ----- > > src/slave/containerizer/docker.cpp 391700f9698d0658b9273d79857bfa30bf3549be > > > Diff: https://reviews.apache.org/r/67585/diff/3/ > > > Testing > ------- > > > Thanks, > > CHEN RUNCONG > >
