cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Branimir Lambov (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-13305) Slice.isEmpty() returns false for some empty slices
Date Wed, 08 Mar 2017 12:38:38 GMT


Branimir Lambov commented on CASSANDRA-13305:

The patch looks good, but the failures in {{RowsTest}} on 3.0 appear to be new. Could you
verify that they are not genuine regressions?

> Slice.isEmpty() returns false for some empty slices
> ---------------------------------------------------
>                 Key: CASSANDRA-13305
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 3.0.x, 3.11.x
> {{Slice.isEmpty}} is currently defined as {{, start) < 0}} but
this shouldn't be a strict inequality. Indeed, the way {{Slice.Bound}} is defined, having
a start equal to an end implies a range like {{[1, 1)}}, but that range is definitively empty
and something we shouldn't let in as that would break merging and other range tombstone related
> In practice, if you can currently insert such empty range (with something like {{DELETE
FROM t WHERE k = 'foo' AND i >= 1 AND i < 1}}), and that can trigger assertions in {{RangeTomstoneList}}
(and possibly other problem).

This message was sent by Atlassian JIRA

View raw message