> On April 3, 2016, 2:22 p.m., Guangya Liu wrote: > > src/slave/containerizer/docker.cpp, lines 825-826 > > <https://reviews.apache.org/r/44823/diff/4/?file=1323359#file1323359line825> > > > > I think that here should be `||` but not `&&` > > haosdent huang wrote: > Actually these code existed before. If change to `||`, those containers > could be found in `existingContainers` and `executorInfo.has_container()` is > false would not be recovered. > > Guangya Liu wrote: > I think that when recover, we should fileter out the continers whose > executor does not include container info or the does not recognized by the > `existingContainers`?
If we filter out all executorInfo that doesn't contain `container()`, we would miss this case: the containerId exists in existingContainers and its `executorInfo` doesn't have container(). - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44823/#review126728 ----------------------------------------------------------- On April 4, 2016, 4:48 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44823/ > ----------------------------------------------------------- > > (Updated April 4, 2016, 4:48 p.m.) > > > Review request for mesos, Benjamin Hindman, Ian Downes, Jie Yu, Till > Toenshoff, and Timothy Chen. > > > Bugs: MESOS-5085 > https://issues.apache.org/jira/browse/MESOS-5085 > > > Repository: mesos > > > Description > ------- > > Remove `SlaveState` in `DockerContainerizer` during recover. > > > Diffs > ----- > > src/slave/containerizer/docker.hpp 89d450e10a84f24ddd46d517e2b4b46ab02c4fda > src/slave/containerizer/docker.cpp 9314d1f9e0b6077fe7c48b860783ab21acc48be6 > src/tests/containerizer/docker_containerizer_tests.cpp > b9c26b92c44e8ca718a5eb333855199bdf2a8e81 > > Diff: https://reviews.apache.org/r/44823/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >
