mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From haosdent huang <haosd...@gmail.com>
Subject Re: Review Request 44762: [WIP][PROPOSAL_1]Add CgroupsIsolator. This is only used for discussion.
Date Sun, 13 Mar 2016 18:23:09 GMT

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




src/slave/containerizer/mesos/isolators/cgroups/info.hpp (line 55)
<https://reviews.apache.org/r/44762/#comment185539>

    Another problem here is could not avoid different threads to modify a same `CgroupsIsolatorInfo`
object in a easy way.


- haosdent huang


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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message