> On March 19, 2016, 12:39 a.m., Jie Yu wrote:
> > src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp, line 363
> > <https://reviews.apache.org/r/44762/diff/1/?file=1296905#file1296905line363>
> >
> >     I think what you can do is to save a future in each Subsystem.
> >     
> >     I am not sure if we have a primitive in libprocess to watch for if any 
> > future is ready or not. We can potentially add that. This function just 
> > watch for all futures in Subsystems and return that if any of them is 
> > ready. If any of them failed/discarded, return the failure.

>I think what you can do is to save a future in each Subsystem.
> This function just watch for all futures in Subsystems and return that if any 
> of them is ready. If any of them failed/discarded, return the failure.

Yes, we do this in [PROPOSAL_2](https://reviews.apache.org/r/44761/)

>I am not sure if we have a primitive in libprocess to watch for if any future 
>is ready or not.

I also think it would be better to add this in libprocess, so that we no need 
use `forloop` to register handlers on every future.


- haosdent


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44762/#review124340
-----------------------------------------------------------


On March 13, 2016, 5:50 p.m., haosdent huang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44762/
> -----------------------------------------------------------
> 
> (Updated March 13, 2016, 5:50 p.m.)
> 
> 
> Review request for mesos and Jie Yu.
> 
> 
> Bugs: MESOS-4697
>     https://issues.apache.org/jira/browse/MESOS-4697
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Add CgroupsIsolator. This is only used for discussion.
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt 0517dd1ae7125ac1ae85cc83b6daaca66a10b8b4 
>   src/Makefile.am f2a592dba3e963c99c48c4b9045b4a04d173cb22 
>   src/slave/containerizer/mesos/isolators/cgroups/cgroups.hpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/cgroups/info.hpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/cgroups/info.cpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/cgroups/subsystem.hpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/44762/diff/
> 
> 
> Testing
> -------
> 
> The relations between classes in this proposal is:
> 
> ```
> +------------------------+
> |                        |
> | CgroupsIsolatorProcess <---------------------+
> |                        |                     |
> +------------^-----------+                     +
>              |                           Belongs to
>              +                                 +
>         Belongs to                             |
>              +                                 |
>              |                                 |
>        +-----+-----+               +-----------+---------+
>        |           |               |                     |
>        | Subsystem <---+Used by+---+ CgroupsIsolatorInfo |
>        |           |               |                     |
>        +-----------+               +---------------------+
> ```
> 
> 
> Thanks,
> 
> haosdent huang
> 
>

Reply via email to