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

Brandon Williams commented on CASSANDRA-19676:
----------------------------------------------

I think 5.0.0 is probably a good place for this; it shouldn't cause a 
regression and is a performance boost, but if it does happen to regress then 
catching it in a new release is the best scenario.

> Stream processing for StorageProxy::updateCoordinatorWriteLatencyTableMetric
> ----------------------------------------------------------------------------
>
>                 Key: CASSANDRA-19676
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-19676
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Legacy/Core
>            Reporter: Sam Lightfoot
>            Assignee: Sam Lightfoot
>            Priority: Normal
>             Fix For: 5.x
>
>         Attachments: image-2024-06-02-17-25-25-071.png
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> On profiling a write-heavy workload (90% writes) using easy-cass-stress, it 
> became very clear StorageProxy::updateCoordinatorWriteLatencyTableMetric was 
> a hot path that ~15% of the CPU cycles of 
> ModificationStatement::executeWithoutCondition were taken up by (see attached 
> async-profiler image).
> We should convert this stream to a simple for loop, as has been discussed 
> recently on the mail list.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to