Skip to content

[CASSANDRA-21020][trunk] Optimize thread local for metrics and tracing#4831

Open
netudima wants to merge 1 commit into
apache:trunkfrom
netudima:CASSANDRA-21020-trunk
Open

[CASSANDRA-21020][trunk] Optimize thread local for metrics and tracing#4831
netudima wants to merge 1 commit into
apache:trunkfrom
netudima:CASSANDRA-21020-trunk

Conversation

@netudima
Copy link
Copy Markdown
Contributor

Implement custom CassandraThread to keep direct references to frequently used thread local objects

patch by Dmitry Konstantinov; reviewed by TBD for CASSANDRA-21020

@netudima netudima force-pushed the CASSANDRA-21020-trunk branch 2 times, most recently from 186e3c7 to 0da4a5e Compare May 21, 2026 15:31
Comment thread src/java/org/apache/cassandra/concurrent/NamedThreadFactory.java
Comment thread src/java/org/apache/cassandra/concurrent/CassandraThread.java Outdated
Comment thread test/long/org/apache/cassandra/db/commitlog/CommitLogStressTest.java Outdated
Comment thread src/java/org/apache/cassandra/concurrent/CassandraThread.java Outdated
Comment thread src/java/org/apache/cassandra/concurrent/CassandraThread.java Outdated
Comment thread src/java/org/apache/cassandra/metrics/ThreadLocalMetrics.java Outdated
Copy link
Copy Markdown
Contributor

@belliottsmith belliottsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 this is great, look forward to rolling it out to all of our commonly accessed thread locals

@netudima
Copy link
Copy Markdown
Contributor Author

netudima commented Jun 2, 2026

I was thinking about BufferPool and TinyPool instances, as one of the next places to apply

@smiklosovic smiklosovic self-requested a review June 2, 2026 09:20
…tly used thread local objects

patch by Dmitry Konstantinov; reviewed by Benedict Elliott Smith, Stefan Miklosovic for CASSANDRA-21020
@netudima netudima force-pushed the CASSANDRA-21020-trunk branch from 49163e2 to e0776b6 Compare June 2, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants