cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksey Yeschenko (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-13953) Switch to CRC32 for sstable metadata checksums
Date Mon, 16 Oct 2017 10:42:00 GMT


Aleksey Yeschenko commented on CASSANDRA-13953:

Pushed a commit with the change [here|].
Jenkins dtest run results to be available [here|],
and circle utests [here|].

Changes made:
1. Simplified {{MetadataSerializer.deserialize()}} method by splitting toc decoding logic
into two loops and removing some nesting/branching in components decoding loop
2. Switched from MD5 to CRC32
3. Added checksums to size and overall toc, so that we don't allocate buffers too big because
of corruption in either size or toc offsets.

In addition to that, removed {{MetadataSerializerTest.testMcReadNa()}}, as there is no expectation
of compat between majors. Also added a few tests (ma read ma, mb read mb, mc read mc, and
na read na) to cover roundrtip ser/deser. Also removed the exception for {{MetadataType.STATS}}
not being compared. With all current formats under test we expect everything to come out equal.

> Switch to CRC32 for sstable metadata checksums
> ----------------------------------------------
>                 Key: CASSANDRA-13953
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Marcus Eriksson
>            Assignee: Aleksey Yeschenko
>             Fix For: 4.x
> We should switch to CRC32 for sstable metadata checksumming for consistency with the
rest of the code base. There are a few other cleanups that should be done at the same time.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message