[CASSANDRA-10153] Refactor SequentialWriter, RandomAccessReader, CompressionMetadata and all related classes

• Type: Improvement
• Status: Open
• Priority: Low
• Resolution: Unresolved
• Fix Version/s: 5.x
• Component/s: Legacy/Local Write-Read Paths
• Assignee: Unassigned
• Reporter: Benedict Elliott Smith
• Created: 21/Aug/15 16:33
• Updated: 07/Mar/23 10:54

Description

These classes have been a bit of a mess for a while, with very poor isolation of concerns and lack of consistent approach. We've done a lot of tidying of behaviour, so I think we're now in a good place to restructure them to more clearly represent that behaviour in the simplest way possible.

In my opinion these should all coexist in a single package, since they are so tightly related. The details should not leak outside of this package as far as possible, so there is no leakage into e.g. SSTableWriter.

At the same time we should, as suggested by Blake, separate SequentialWriter from its Transactional behaviour.

Issue Links

• relates to: CASSANDRA-10158 Rationalize implementations of DataInputPlus and DataOutputPlus