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

Paulo Motta commented on CASSANDRA-14082:
-----------------------------------------

Currently the scrubber and relocate sstables were relying on the compaction 
strategy index, so this patches change these operation to use a 
{{DiskBoundaries}} object instead and make {{CSM.getCompactionStrategyIndex}} 
private.

* [3.11 patch|https://github.com/pauloricardomg/cassandra/tree/3.11-14082]

Since this depends on CASSANDRA-13948, I will wait until that is committed 
before setting this as PA.

> Do not expose compaction strategy index publicly
> ------------------------------------------------
>
>                 Key: CASSANDRA-14082
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-14082
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Paulo Motta
>            Assignee: Paulo Motta
>
> Before CASSANDRA-13215 we used the compaction strategy index to decide which 
> disk to place a given sstable, but now we can get this directly from the disk 
> boundary manager and keep the compaction strategy index internal only.
> This will ensure external consumers will use a consistent {{DiskBoundaries}} 
> object to perform operations on multiple disks, rather than risking getting 
> inconsistent indexes if the compaction strategy indexes change between 
> successive calls to {{CSM.getCompactionStrategyIndex}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to