[ https://issues.apache.org/jira/browse/YARN-3513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14532905#comment-14532905 ]
Naganarasimha G R commented on YARN-3513: ----------------------------------------- Checkstyle is not completely related to this patch modifications and test cases are not there as modifications are as part of logging. Can [~djp] or [~vinodkv] further review this patch ? > Remove unused variables in ContainersMonitorImpl and add debug log for > overall resource usage by all containers > ---------------------------------------------------------------------------------------------------------------- > > Key: YARN-3513 > URL: https://issues.apache.org/jira/browse/YARN-3513 > Project: Hadoop YARN > Issue Type: Bug > Components: nodemanager > Reporter: Naganarasimha G R > Assignee: Naganarasimha G R > Priority: Trivial > Labels: BB2015-05-TBR, newbie > Attachments: YARN-3513.20150421-1.patch, YARN-3513.20150503-1.patch, > YARN-3513.20150506-1.patch, YARN-3513.20150507-1.patch > > > Some local variables in MonitoringThread.run() : {{vmemStillInUsage and > pmemStillInUsage}} are not used and just updated. > Instead we need to add debug log for overall resource usage by all containers -- This message was sent by Atlassian JIRA (v6.3.4#6332)