[CASSANDRA-9067] BloomFilter serialization format should not change byte ordering

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 4.0-alpha1, 4.0
• Assignee: Jay Zhuang
• Reporter: Benedict Elliott Smith

Authors

Jay Zhuang

Reviewers

Jason Brown

• Created: 29/Mar/15 15:28
• Updated: 10/Jun/20 10:18
• Resolved: 19/Jan/18 14:43

Description

As a follow-up to CASSANDRA-9066 and CASSANDRA-9060, it appears we do some unnecessary byte swapping during the serialization of bloom filters, which makes the logic slower and harder to follow. We should either perform them more efficiently (using Long.reverseBytes) or, preferably, eliminate the conversion altogether since it does not appear to serve any purpose.

Issue Links

• relates to: CASSANDRA-15869 MemoryOutputStream overflow on large bloom filter

Sub-Tasks

• Remove unused on-heap BloomFilter implementation: Resolved