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

Sylvain Lebresne commented on CASSANDRA-9799:
---------------------------------------------

Committed with the randomness re-enabled, thanks.

bq. Side note: all the methods in RangeTombstoneList.Serializer are //TODO - 
what are our plans for addressing that?

That was for CASSANDRA-9704 (which is incoming) since we only need to serialize 
that for backward compatibility. That said, CASSANDRA-9704 doesn't actually use 
it (that is, it actually implement the equivalent in {{LegacyLayout}} where 
most of the compat code is) so I've remove the Serializer to avoid keeping dead 
code around.

> RangeTombstonListTest sometimes fails on trunk
> ----------------------------------------------
>
>                 Key: CASSANDRA-9799
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9799
>             Project: Cassandra
>          Issue Type: Test
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>              Labels: test
>             Fix For: 3.0 beta 1
>
>
> I've seen random failures with {{RangeTombstoneList.addAllRandomTest}}. The 
> problem is 2 inequalities in {{RangeTombstoneList.insertFrom}} that should be 
> inclusive rather than strict when we deal with boundaries between range. In 
> practice, that makes us consider range like {{[3, 3)}} during addition, which 
> is non-sensical.
> Attaching patch as well as a test that reproduce (extracted from 
> {{addAllRandomTest}} with a failing seed).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to