Re: Review Request 47212: Removed duplicate call to containerizer::executorEnvironment.

2016-05-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47212/#review135123 --- Ship it! Ship It! - Jie Yu On May 11, 2016, 2:13 a.m.,

Re: Review Request 47212: Removed duplicate call to containerizer::executorEnvironment.

2016-05-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47212/#review135106 --- src/slave/containerizer/docker.cpp (line 1195)

Re: Review Request 47212: Removed duplicate call to containerizer::executorEnvironment.

2016-05-24 Thread Joseph Wu
> On May 23, 2016, 8:10 p.m., Kapil Arya wrote: > > src/slave/containerizer/docker.cpp, line 1195 > > > > > > Would there be any difference in the computed environment here vs in > > container constructor? The

Re: Review Request 47212: Removed duplicate call to containerizer::executorEnvironment.

2016-05-23 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47212/#review134500 --- src/slave/containerizer/docker.cpp (line 1195)

Review Request 47212: Removed duplicate call to containerizer::executorEnvironment.

2016-05-10 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47212/ --- Review request for mesos, Adam B, Artem Harutyunyan, Jie Yu, and Kapil Arya.