Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-20 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80845 --- Patch looks great! Reviews applied: [33257] All tests passed. - M

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-20 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/ --- (Updated April 20, 2015, 9:18 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-20 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80813 --- src/slave/containerizer/mesos/containerizer.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-20 Thread Timothy Chen
> On April 18, 2015, 2:55 a.m., Till Toenshoff wrote: > > src/slave/containerizer/docker.cpp, line 424 > > > > > > Not sure what would be correct but I guess it would be right to use a > > lower-case "d" for "docker"

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-20 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80756 --- src/slave/containerizer/docker.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-17 Thread Till Toenshoff
> On April 18, 2015, 2:55 a.m., Till Toenshoff wrote: > > One more tiny nit, the summary has a typo. - Till --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80576

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-17 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80576 --- Ship it! src/slave/containerizer/docker.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-17 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80512 --- Patch looks great! Reviews applied: [33257] All tests passed. - M

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-17 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/ --- (Updated April 17, 2015, 7:07 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-17 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80504 --- src/slave/containerizer/docker.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80321 --- src/slave/slave.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-16 Thread Erik Weathers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/#review80308 --- src/slave/containerizer/docker.cpp

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-16 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/ --- (Updated April 16, 2015, 7:10 a.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-16 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/ --- (Updated April 16, 2015, 7:08 a.m.) Review request for mesos, Benjamin Hindman,

Review Request 33257: Fixed recover tasks only by the intiated containerizer.

2015-04-16 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33257/ --- Review request for mesos, Benjamin Hindman, Bernd Mathiske, Ian Downes, Jie Yu,