[ 
https://issues.apache.org/jira/browse/CASSANDRA-20927?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever updated CASSANDRA-20927:
-------------------------------------------
    Description: 
Range is left exclusive. When constructing Range, it should decrement left 
position instead of incrementing right position.

Also ShardManager#rangeSpanned for (min, min] should be 1 instead of 1 token 
span.

Impact is trivial: it can cause is a range to be measured to span 0 range 
instead of 2^-64 (if it only contains one partition that falls on a local 
ownership boundary), but this is then anyway corrected to give it a sensible 
density. 

The min-min case only relates to tests.

downstream patch: 
https://github.com/datastax/cassandra/commit/dd5da490b5705a61b49bc6bfadf6a47a973c9804

  was:
Range is left exclusive. When constructing Range, it should decrement left 
position instead of incrementing right position.

Also ShardManager#rangeSpanned for (min, min] should be 1 instead of 1 token 
span.

Impact is trivial: it can cause is a range to be measured to span 0 range 
instead of 2^-64 (if it only contains one partition that falls on a local 
ownership boundary), but this is then anyway corrected to give it a sensible 
density. 

The min-min case only relates to tests.


> Fix ShardManager#coveringRange to include provide partition position
> --------------------------------------------------------------------
>
>                 Key: CASSANDRA-20927
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-20927
>             Project: Apache Cassandra
>          Issue Type: Bug
>          Components: Local/Compaction
>            Reporter: Michael Semb Wever
>            Priority: Normal
>             Fix For: 5.0.x, 5.x
>
>
> Range is left exclusive. When constructing Range, it should decrement left 
> position instead of incrementing right position.
> Also ShardManager#rangeSpanned for (min, min] should be 1 instead of 1 token 
> span.
> Impact is trivial: it can cause is a range to be measured to span 0 range 
> instead of 2^-64 (if it only contains one partition that falls on a local 
> ownership boundary), but this is then anyway corrected to give it a sensible 
> density. 
> The min-min case only relates to tests.
> downstream patch: 
> https://github.com/datastax/cassandra/commit/dd5da490b5705a61b49bc6bfadf6a47a973c9804



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to