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)