> On June 12, 2016, 3:18 p.m., haosdent huang wrote: > > `src/files/files.cpp` have similar problems. > > > > In discription, `differen` shoud be `different`? > > Joerg Schad wrote: > If you mean > https://github.com/apache/mesos/blob/master/src/files/files.cpp#L591 that is > on my investigation list, > all other occurences in that file shoud have been fixed with > https://github.com/apache/mesos/commit/97124ac72bbc42f809b800fe2da383dc43b1b34e, > which was committed this morning. > Thanks for pointing out the typo!
Oh, sorry, my bad, I have not yet updated my local repo. - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48599/#review137172 ----------------------------------------------------------- On June 12, 2016, 4:05 p.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48599/ > ----------------------------------------------------------- > > (Updated June 12, 2016, 4:05 p.m.) > > > Review request for mesos, Jie Yu and Joseph Wu. > > > Bugs: MESOS-5604 > https://issues.apache.org/jira/browse/MESOS-5604 > > > Repository: mesos > > > Description > ------- > > Previously the continuation followed via \`.then(\[=\]\`, > which potentially executes the continuation on a different > process. This patch fixes this behavior (it should > run on the same process) and avoids potential race > conditions. > > > Diffs > ----- > > src/slave/containerizer/docker.cpp 1af5b451922e1eeb0af025f29c53a85ab9deec3b > > Diff: https://reviews.apache.org/r/48599/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Joerg Schad > >
