[ 
https://issues.apache.org/jira/browse/HDFS-8116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14534163#comment-14534163
 ] 

Brahma Reddy Battula commented on HDFS-8116:
--------------------------------------------

I think, Performance test might not require..Just for consistency I removed 
LOG.isDebugEnabled().. If I add LOG.isDebugEnabled() before LOG.Debug() ( as 
per defect summary),Then we need to do performance test before and after the 
patch...Please correct me , If I am wrong.. Need to change defect summary..

> RollingWindowManager#getTopUserForMetric should check if LOG.isDebugEnabled() 
> before LOG.debug() 
> -------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-8116
>                 URL: https://issues.apache.org/jira/browse/HDFS-8116
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode
>    Affects Versions: 2.7.0
>            Reporter: Xiaoyu Yao
>            Assignee: Brahma Reddy Battula
>            Priority: Trivial
>              Labels: BB2015-05-TBR
>         Attachments: HDFS-8116.patch
>
>
> RollingWindowManager#getTopUserForMetric should check if LOG.isDebugEnabled() 
> before LOG.debug() 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to