You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Andy Tolbert (JIRA)" <ji...@apache.org> on 2015/08/25 22:00:46 UTC
[jira] [Commented] (CASSANDRA-10183) logger.trace statements in hot
code path in CodecRegistry
[ https://issues.apache.org/jira/browse/CASSANDRA-10183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14711870#comment-14711870 ]
Andy Tolbert commented on CASSANDRA-10183:
------------------------------------------
My mistake, on the wrong JIRA, this should be a driver issue.
> logger.trace statements in hot code path in CodecRegistry
> ---------------------------------------------------------
>
> Key: CASSANDRA-10183
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10183
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Andy Tolbert
> Priority: Minor
>
> There are a number of logger.trace statements in CodecRegistry that should be updated to log conditionally based on whether or not trace is enabled, i.e.:
> {code:java}
> logger.trace("Querying cache for codec [{} <-> {}]", cqlType, javaType);
> {code}
> should be:
> {code:java}
> if(logger.isTraceEnabled())
> logger.trace("Querying cache for codec [{} <-> {}]", cqlType, javaType);
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)