[ 
https://issues.apache.org/jira/browse/CASSANDRA-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12719579#action_12719579
 ] 

Jonathan Ellis commented on CASSANDRA-230:
------------------------------------------

I think removing ChecksumManager and ChecksumRandomAccessFile is a better fix, 
because
 1. they are unused
 2. CRAF is a copy/paste modification of BufferedRAF
 3. checksum-per-block is the wrong approach; we want checksum-per-column (and 
per-CF-index) instead so we can checksum reads easily

> Race in ChecksumManager.instance()
> ----------------------------------
>
>                 Key: CASSANDRA-230
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-230
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: trunk
>            Reporter: Stu Hood
>         Attachments: 230.patch, checksum-manager-race.patch
>
>
> There is a minor race condition in ChecksumManager.instance(). Patch attached.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to