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

Reply via email to