[
https://issues.apache.org/jira/browse/MESOS-6035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15425251#comment-15425251
]
Yan Xu edited comment on MESOS-6035 at 8/22/16 5:35 PM:
--------------------------------------------------------
I commented on the review.
This is relevant to MESOS-5879 as well. [~swsnider] brought up a point if we
default to *not* recursively descend into nested cgroups, a lot of problems
including MESOS-5879 go aways. I can't think of a reason that we should
traverse the cgroups recursively today (except {{cgroups::remove()}} and
{{cgroups::destroy()}} but they are different in that they are within the
cgroups util and encapsulate such details). Of course if we do have cases in
the future they can set {{recursive=true}} explicitly.
[~jieyu] [~idownes] what are your thoughts on this?
was (Author: xujyan):
I commented on the review.
This is relevant to MESOS-5879 as well. [~swsnider] brought up a point if we
default to *not* recursively descend into nested cgroups, a lot of problems
including MESOS-5879 go aways. I can't think of a reason that we should
traverse the cgroups recursively today. Of course if we do have cases in the
future they can set {{recursive=true}} explicitly.
[~jieyu] [~idownes] what are your thoughts on this?
> Add non-recursive version of cgroups::get
> -----------------------------------------
>
> Key: MESOS-6035
> URL: https://issues.apache.org/jira/browse/MESOS-6035
> Project: Mesos
> Issue Type: Improvement
> Reporter: haosdent
> Assignee: haosdent
> Priority: Minor
>
> In some cases, we only need to get the top level cgroups instead of to get
> all cgroups recursively. Add a non-recursive version could help to avoid
> unnecessary paths.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)