[CASSANDRA-17998] Mutation Serialization Caching

• Type: Improvement
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 5.0-alpha1, 5.0
• Component/s: Messaging/Internode
• Change Category: Performance
• Complexity: Challenging
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/227409d9201fa1aeb9f80b22f499577aedfe25bc

Test and Documentation Plan

CI

• Assignee: T Jake Luciani
• Reporter: T Jake Luciani

Authors

T Jake Luciani

Reviewers

Josh McKenzie

• Created: 26/Oct/22 21:11
• Updated: 12/Sep/23 13:02
• Resolved: 19/Dec/22 21:24

Description

A performance change that adds mutation serialization caching to avoid re-serializing the mutation for commitlog and nodes twice. 

• Cached serialization for storage proxy and local commitlog
• Cached deserialization for messaging service and local commitlog

This yields a non trivial perf gain (~7-10%) and latency drop (median)

https://tjake.github.io/other/cached-mutations-report.html

 

The cached buffer is stored by MessagingService version to avoid being used by differing nodes during upgrades.

Also, It avoids caching mutations larger than a threshold to avoid GC issues. 

 

GH PR: https://github.com/apache/cassandra/pull/1954