cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-5492) Backport row-level bloom filter removal to 1.2
Date Fri, 19 Apr 2013 18:57:17 GMT


Jonathan Ellis commented on CASSANDRA-5492:

Pushed to

Second commit adds ScrubTest back, minus testScrubFile that appears to have relied on a bogus
bloom filter, which is now skipped on the read path.
> Backport row-level bloom filter removal to 1.2
> ----------------------------------------------
>                 Key: CASSANDRA-5492
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 1.2.0
>            Reporter: Jonathan Ellis
>            Assignee: Jonathan Ellis
>            Priority: Minor
>             Fix For: 1.2.5
> With the possible presence of range tombstones, it is erroneous to skip checking for
a given column in SSTableNamesIterator because the bloom filter says it is not there.
> This is fixed by CASSANDRA-5487, which ignores the BF, but it's a shame to leave unused
bloom filters cluttering up the row cache.
> If it's too risky to do a full backport we can just substitute an AlwaysPresentFilter
in RIE.create, but I think the code removal is fairly straightforward.

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