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

David Capwell commented on CASSANDRA-17575:
-------------------------------------------

sorry never got the time to work on this.

[~brandon.williams] the test linked, if you change to do (32, 31] you see that 
compaction included a partition it shouldn't have; we include partition 32 even 
though the range filters out that partition.  This ticket should figure out why 
we include the wrong partition and attempt to fix it so we stop

> forceCompactionForTokenRange when using a wrapped range may include sstables 
> not within that range
> --------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-17575
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17575
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local/Compaction
>            Reporter: David Capwell
>            Priority: Normal
>             Fix For: 4.1-beta, 4.1.x
>
>
> This was found in CASSANDRA-17537
> When you compact the range (32, 31] this should include everything BUT 32, 
> but in the test 
> org.apache.cassandra.db.compaction.LeveledCompactionStrategyTest#testTokenRangeCompaction
>  it found that SSTables with the bounds (32, 32) were getting included in the 
> set of SSTables to compact



--
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