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

Reply via email to