[
https://issues.apache.org/jira/browse/CASSANDRA-18836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17777425#comment-17777425
]
Caleb Rackliffe commented on CASSANDRA-18836:
---------------------------------------------
Finished my first pass at review and left a few comments. Once we're happy w/
how we've dealt w/ those, I'll approve both patches. (\{{trunk}} patch looks
identical to 5.0)
> Replace CRC32 w/ CRC32C in IndexFileUtils.ChecksummingWriter
> ------------------------------------------------------------
>
> Key: CASSANDRA-18836
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18836
> Project: Cassandra
> Issue Type: Improvement
> Components: Feature/2i Index, Feature/SAI
> Reporter: Caleb Rackliffe
> Assignee: Maxim Muzafarov
> Priority: Normal
> Fix For: 5.0-alpha2, 5.x
>
> Time Spent: 2h
> Remaining Estimate: 0h
>
> It seems that now we're on Java 11 for 5.0, there isn't much reason not to
> use CRC32C as a drop-in replacement for CRC32. SAI isn't even released, so
> has no binary compatibility entanglements, and this should be pretty
> straightforward.
> See https://github.com/apache/bookkeeper/pull/3309
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
