hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-6668) Use cgroup to get container resource utilization
Date Sat, 24 Jun 2017 03:40:00 GMT

    [ https://issues.apache.org/jira/browse/YARN-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16061777#comment-16061777
] 

ASF GitHub Bot commented on YARN-6668:
--------------------------------------

Github user xshaun commented on a diff in the pull request:

    https://github.com/apache/hadoop/pull/227#discussion_r123869480
  
    --- Diff: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java
---
    @@ -60,6 +60,7 @@
     import org.apache.hadoop.yarn.server.api.records.NodeHealthStatus;
     import org.apache.hadoop.yarn.server.nodemanager.containermanager.ContainerManager;
     import org.apache.hadoop.yarn.server.nodemanager.collectormanager.NMCollectorService;
    +import org.apache.hadoop.yarn.server.api.records.OverAllocationInfo;
    --- End diff --
    
    Is it nice to insert this line between 60th and 61th line ? like these:
    ```
    import org.apache.hadoop.yarn.server.api.records.NodeHealthStatus;
    import org.apache.hadoop.yarn.server.api.records.OverAllocationInfo;
    import org.apache.hadoop.yarn.server.nodemanager.containermanager.ContainerManager;
    ```


> Use cgroup to get container resource utilization
> ------------------------------------------------
>
>                 Key: YARN-6668
>                 URL: https://issues.apache.org/jira/browse/YARN-6668
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>    Affects Versions: 3.0.0-alpha3
>            Reporter: Haibo Chen
>            Assignee: Miklos Szegedi
>         Attachments: YARN-6668.000.patch, YARN-6668.001.patch
>
>
> Container Monitor relies on proc file system to get container resource utilization, which
is not as efficient as reading cgroup accounting. We should in NM, when cgroup is enabled,
read cgroup stats instead. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org


Mime
View raw message