----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26436/#review55955 -----------------------------------------------------------
src/slave/containerizer/docker.cpp <https://reviews.apache.org/r/26436/#comment96314> Can we make the container here `const`? We don't actually do any modification on it in this function. - Michael Park On Oct. 8, 2014, 6:43 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26436/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2014, 6:43 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Repository: mesos-git > > > Description > ------- > > Review: https://reviews.apache.org/r/26436 > > > Diffs > ----- > > src/slave/containerizer/docker.cpp 9a2948951f57f3ab16291df51cd9f33e5e96add4 > > Diff: https://reviews.apache.org/r/26436/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >