[
https://issues.apache.org/jira/browse/HBASE-12061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14145241#comment-14145241
]
Vladimir Rodionov commented on HBASE-12061:
-------------------------------------------
Yes, most places (on critical paths) for DEBUG/TRACE are already wrapped. Do
not have stats, but if they are not wrapped - they are not on critical path.
This is mostly for INFO and for some rare cases (which are still possible)
where we do not have wrappers for DEBUG/TRACE but should.
> Ensure LOG.isXXXEnabled() wrapper around LOG.XXX() calls
> --------------------------------------------------------
>
> Key: HBASE-12061
> URL: https://issues.apache.org/jira/browse/HBASE-12061
> Project: HBase
> Issue Type: Sub-task
> Components: Performance
> Reporter: Vladimir Rodionov
> Assignee: Vladimir Rodionov
>
> The common anti-pattern in Java is using LOG.xxx() calls w/o wrapping them
> into LOG.isXXXEnabled(). This results in unnecessary object serializations/
> string concatenations/ garbage object production even if XXX log level is
> disabled.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
