cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-3872) Sub-columns removal is broken in 1.1
Date Tue, 07 Feb 2012 23:54:59 GMT

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

Jonathan Ellis commented on CASSANDRA-3872:
-------------------------------------------

Could we do a simpler fix by turning column.mostRecentLiveChangeAt into column.mostRecentChangeAt?
I don't think restricting to live columns is actually useful here (although it is in SQF,
so we'd need a new method instead of replacing mRLCA).
                
> Sub-columns removal is broken in 1.1
> ------------------------------------
>
>                 Key: CASSANDRA-3872
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3872
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 1.1
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 1.1
>
>         Attachments: 3872.patch
>
>
> CASSANDRA-3716 actually broke sub-columns deletion. The reason is that in QueryFilter.isRelevant,
we've switched in checking getLocalDeletionTime() only (without looking for isMarkedForDelete).
But for columns containers (in this case SuperColumn), the default local deletion time when
not deleted is Integer.MIN_VALUE. In other words, a SC with only non-gcable tombstones will
be considered as not relevant (while it should).
> This is caught by two unit tests (RemoveSuperColumnTest and RemoveSubColumnTest) that
are failing currently.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message