hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kumar Vavilapalli (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-3513) Remove unused variables in ContainersMonitorImpl
Date Mon, 04 May 2015 21:15:07 GMT

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

Mime
View raw message