cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-5862) Switch to adler checksum for sstables
Date Thu, 08 Aug 2013 20:42:47 GMT


Jonathan Ellis commented on CASSANDRA-5862:

Radim: hardware CRC32C is exactly what Jake tested above.

Jake: the dual checksum fields in CRAR are a bit clunky; the approach of instantiating the
correct one in the constructor that you use in CIS is cleaner.

Nit: if we can come up with a single name to represent "adler and post-compression" then I'd
prefer that to passing two booleans around that are always the same value.

Otherwise LGTM.
> Switch to adler checksum for sstables
> -------------------------------------
>                 Key: CASSANDRA-5862
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: T Jake Luciani
>             Fix For: 2.0.1
>         Attachments: 5862.txt
> Adler is significantly faster than CRC32:
> (Adler is weaker for short inputs, so we should leave the commitlog alone, as it checksums
each mutation individually.)

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message