cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CASSANDRA-230) Race in ChecksumManager.instance()
Date Mon, 15 Jun 2009 14:58:07 GMT

    [ 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.


Mime
View raw message