[ https://issues.apache.org/jira/browse/YARN-3513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14527317#comment-14527317 ]
Vinod Kumar Vavilapalli commented on YARN-3513: ----------------------------------------------- bq. This may be fine in trunk, but since YARN-2928 needs to merge back in near future, we may not want to make the change on content for now. We need to consider code clean up comprehensively when we're doing the branch merge. Is there agreement on this? If so, let's commit this (and any other change required) to YARN-2928 first? > Remove unused variables in ContainersMonitorImpl > ------------------------------------------------ > > 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: newbie > Attachments: YARN-3513.20150421-1.patch, YARN-3513.20150503-1.patch > > > class members : {{private final Context context;}} > and some local variables in MonitoringThread.run() : {{vmemStillInUsage and > pmemStillInUsage}} are not used and just updated. -- This message was sent by Atlassian JIRA (v6.3.4#6332)