Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-03-01 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/ --- (Updated March 1, 2016, 10:17 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-03-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/#review121484 --- src/slave/containerizer/mesos/isolators/cgroups/net_cls.cpp

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-03-01 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/ --- (Updated March 1, 2016, 7:58 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-03-01 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/ --- (Updated March 1, 2016, 7:55 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-03-01 Thread Avinash sridharan
> On Feb. 29, 2016, 10:58 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/cgroups/net_cls.cpp, lines 125-127 > > > > > > I think you should be able to use `foreach` here The problem here is that we

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-02-29 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/#review121339 --- Fix it, then Ship it!

Re: Review Request 43914: Modified `NetClsHandleManager` to take a range of secondary handles.

2016-02-24 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43914/ --- (Updated Feb. 24, 2016, 4:44 p.m.) Review request for mesos and Jie Yu.