Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-26 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review125512 --- Fix it, then Ship it!

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review123473 --- Patch looks great! Reviews applied: [44790, 44791, 44087]

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-14 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 14, 2016, 3:44 p.m.) Review request for mesos and Joris Van

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-14 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 14, 2016, 3:43 p.m.) Review request for mesos and Joris Van

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review123125 --- Patch looks great! Reviews applied: [44247, 44087] Passed

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-11 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 11, 2016, 8:39 a.m.) Review request for mesos and Joris Van

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-11 Thread Joerg Schad
> On March 10, 2016, 8:47 p.m., Joris Van Remoortere wrote: > > src/slave/containerizer/mesos/linux_launcher.cpp, line 301 > > > > > > We now rely on the hook logic to do this right? > > Let's document this.

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-10 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review122977 --- src/slave/containerizer/mesos/linux_launcher.cpp (lines 247 -

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review122182 --- Patch looks great! Reviews applied: [44247, 44087] Passed

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-04 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review122154 --- Ship it! Ship It! - Avinash sridharan On March 3, 2016,

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 3, 2016, 3:39 p.m.) Review request for mesos and Joris Van

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 3, 2016, 8:12 a.m.) Review request for mesos and Joris Van

Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/#review121616 --- Patch looks great! Reviews applied: [44247, 44087] Passed