Re: Review Request 35566: Refactor executorEnvironment to take slave::Flags.

2015-06-18 Thread Benjamin Hindman
On June 18, 2015, 1:01 a.m., Till Toenshoff wrote: src/tests/containerizer.cpp, line 124 https://reviews.apache.org/r/35566/diff/1/?file=986478#file986478line124 This indention will very soon not comply with our styleguide anymore, no? No, it will, we still wrap the same way for

Re: Review Request 35566: Refactor executorEnvironment to take slave::Flags.

2015-06-17 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35566/#review88320 --- Ship it! src/slave/containerizer/containerizer.hpp (lines 132 -

Review Request 35566: Refactor executorEnvironment to take slave::Flags.

2015-06-17 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35566/ --- Review request for mesos and Till Toenshoff. Bugs: MESOS-2832