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

Joshua McKenzie commented on CASSANDRA-9799:
--------------------------------------------

Looks like you left the seed as non-random in 
RangeTombstoneListTest.addAllRandomTest():
{code}
long seed = 886061465634L;
//long seed = System.nanoTime();
{code}

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

CI vs. trunk jobs on the 14th looks good.

+1 w/the above change.

> 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