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

Gera Shegalov commented on HADOOP-12912:
----------------------------------------

isDebugEnabled is worth doing for "best practices" reason imo. I just noticed 
it in a thread dump. It's much more than a branching instruction. there a bunch 
of invokevirtual in commons and log4j logging and invokeinterface call behind 
it.

> Add LOG.isDebugEnabled() guard in Progress.set method
> -----------------------------------------------------
>
>                 Key: HADOOP-12912
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12912
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Tsuyoshi Ozawa
>            Assignee: Tsuyoshi Ozawa
>         Attachments: HADOOP-12912.001.patch
>
>




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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to