cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-5625) Cleanup Tracing class
Date Fri, 21 Jun 2013 14:17:20 GMT

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

Jonathan Ellis commented on CASSANDRA-5625:
-------------------------------------------

I thought you were going to update it after my comments. :P
                
> Cleanup Tracing class
> ---------------------
>
>                 Key: CASSANDRA-5625
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5625
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Mikhail Mazursky
>            Priority: Trivial
>         Attachments: trunk-5625.patch
>
>
> 1. Tracing instance cannot be null because it's initialized at classloading time and
never changes. Made it final to prove that. Removed all redundant null checks.
> 2. StorageService has a Tracing.instance() call - it should trace something or can be
removed?
> 3. Made sessions a ConcurrentMap to stress that it should be thread-safe.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message