[CASSANDRA-10193] Improve Rows.diff and Rows.merge efficiency; downgrade Row.columns() to Collection<ColumnDefinition>

• Type: Improvement
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 3.0 beta 2
• Assignee: Benedict Elliott Smith
• Reporter: Benedict Elliott Smith

Authors

Benedict Elliott Smith

Reviewers

Marcus Eriksson

• Created: 26/Aug/15 08:23
• Updated: 16/Apr/19 09:30
• Resolved: 04/Sep/15 13:02

Description

There's not really a lot of reason to store a Columns instance in each row. Retaining it introduces extra costs on every row merge, in both consumed CPU time and heap.

While working on CASSANDRA-10045 it became apparent this would be very easy to remove, however to avoid scope creep I have filed this as a follow up ticket.

Issue Links

• is duplicated by: CASSANDRA-10169 Test regression for consistency_test.TestAccuracy
• is duplicated by: CASSANDRA-10192 Rows.merge is inefficient