> On Jan. 9, 2015, 2:50 p.m., Bernd Mathiske wrote: > > src/slave/containerizer/docker.cpp, line 313 > > <https://reviews.apache.org/r/29327/diff/1/?file=798947#file798947line313> > > > > This was below 80 chars, was it not?
It is, but BenH wants all the style to be consistent when it comes to dispatching (he made a similar comment in another file in another review before). So either they're all one-line, or all multi. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29327/#review67437 ----------------------------------------------------------- On Jan. 6, 2015, 10:14 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29327/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 10:14 p.m.) > > > Review request for mesos, Benjamin Hindman and Bernd Mathiske. > > > Repository: mesos-git > > > Description > ------- > > Add slave id to docker container name prefix. > > > Diffs > ----- > > src/slave/containerizer/docker.hpp 28ebc6272cd68167fc9a0898fd8eb9d53890b815 > src/slave/containerizer/docker.cpp 19a6ea2b5342abe4bf10cf4ea2d5d756df9f8665 > > Diff: https://reviews.apache.org/r/29327/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >
