[CASSANDRA-5625] Cleanup Tracing class

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 2.0 beta 1
• Assignee: Mikhail Mazursky
• Reporter: Mikhail Mazursky

Authors

Mikhail Mazursky

Reviewers

Jonathan Ellis

• Created: 09/Jun/13 09:30
• Updated: 16/Apr/19 09:32
• Resolved: 24/Jun/13 13:45

Description

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.