[jira] [Updated] (CASSANDRA-14082) Do not expose compaction strategy index publicly

2017-12-26 Thread Paulo Motta (JIRA)

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

Paulo Motta updated CASSANDRA-14082:

   Resolution: Fixed
Fix Version/s: 4.0
   3.11.2
   Status: Resolved  (was: Ready to Commit)

Committed as {{e208a6a210d172b991b40fb66a4763e30b3e4d7d}} to cassandra-3.11 
branch and merged up to master. Thanks!

> 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
> Fix For: 3.11.2, 4.0
>
> Attachments: 3.11-14082-dtest.png, 3.11-14082-testall.png, 
> trunk-14082-dtest.png, trunk-14082-testall.png
>
>
> 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



[jira] [Updated] (CASSANDRA-14082) Do not expose compaction strategy index publicly

2017-12-22 Thread Marcus Eriksson (JIRA)

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

Marcus Eriksson updated CASSANDRA-14082:

Status: Ready to Commit  (was: Patch Available)

> 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
> Attachments: 3.11-14082-dtest.png, 3.11-14082-testall.png, 
> trunk-14082-dtest.png, trunk-14082-testall.png
>
>
> 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



[jira] [Updated] (CASSANDRA-14082) Do not expose compaction strategy index publicly

2017-12-14 Thread Paulo Motta (JIRA)

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

Paulo Motta updated CASSANDRA-14082:

Status: Patch Available  (was: Open)

Rebased on top of latest now that CASSANDRA-13948, setting this to PA, tests 
look good:

||3.11||trunk||
|[branch|https://github.com/apache/cassandra/compare/cassandra-3.11...pauloricardomg:3.11-14082]|[branch|https://github.com/apache/cassandra/compare/trunk...pauloricardomg:trunk-14082]|
|[testall|https://issues.apache.org/jira/secure/attachment/12902160/3.11-14082-testall.png]|[testall|https://issues.apache.org/jira/secure/attachment/12902158/trunk-14082-testall.png]|
|[dtest|https://issues.apache.org/jira/secure/attachment/12902161/3.11-14082-dtest.png]|[dtest|https://issues.apache.org/jira/secure/attachment/12902159/trunk-14082-dtest.png]|

> 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
> Attachments: 3.11-14082-dtest.png, 3.11-14082-testall.png, 
> trunk-14082-dtest.png, trunk-14082-testall.png
>
>
> 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



[jira] [Updated] (CASSANDRA-14082) Do not expose compaction strategy index publicly

2017-12-14 Thread Paulo Motta (JIRA)

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

Paulo Motta updated CASSANDRA-14082:

Attachment: 3.11-14082-dtest.png
3.11-14082-testall.png
trunk-14082-dtest.png
trunk-14082-testall.png

> 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
> Attachments: 3.11-14082-dtest.png, 3.11-14082-testall.png, 
> trunk-14082-dtest.png, trunk-14082-testall.png
>
>
> 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