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

Siddharth Seth commented on TEZ-2225:
-------------------------------------

The source string will always be created if we remove the isDebugEnabled checks 
? Is the JVM taking care of interning them - or maybe slf4j itself ?
I'd be especially vary about debug statements which are in a loop.

> Remove instances of LOG.isDebugEnabled
> --------------------------------------
>
>                 Key: TEZ-2225
>                 URL: https://issues.apache.org/jira/browse/TEZ-2225
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Vasanth kumar RJ
>            Assignee: Vasanth kumar RJ
>            Priority: Minor
>              Labels: performance
>         Attachments: TEZ-2225.1.patch, TEZ-2225.2.patch, TEZ-2225.3.patch
>
>
> Remove LOG.isDebugEnabled() and use parameterized debug logging



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

Reply via email to