----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30509/#review71118 -----------------------------------------------------------
src/slave/containerizer/mesos/containerizer.hpp <https://reviews.apache.org/r/30509/#comment116727> I wonder if we should consolidate our terminalogy for the working directory. I've been calling it sandbox in the docker containerizer, and sometimes it's referred to workDir, and here just directory. Can we just call this sandbox? - Timothy Chen On Feb. 4, 2015, 10:56 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30509/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2015, 10:56 p.m.) > > > Review request for mesos, Ben Mahler, Ian Downes, Michael Park, and Vinod > Kone. > > > Bugs: MESOS-2031 > https://issues.apache.org/jira/browse/MESOS-2031 > > > Repository: mesos > > > Description > ------- > > Added executor working directory to Container struct in Mesos containerizer. > > The executor directory is needed when linking persistent volumes inside the > container. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.hpp > 569119fc591ecb118ecff51b8b19d0a1b8090cb5 > src/slave/containerizer/mesos/containerizer.cpp > d712278428889ebdfd598537690138329d8464f0 > > Diff: https://reviews.apache.org/r/30509/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
