You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Aleksey Yeschenko (JIRA)" <ji...@apache.org> on 2015/06/04 20:21:40 UTC
[jira] [Updated] (CASSANDRA-9314) Overload SecondaryIndex#indexes
to accept the column definition
[ https://issues.apache.org/jira/browse/CASSANDRA-9314?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Aleksey Yeschenko updated CASSANDRA-9314:
-----------------------------------------
Fix Version/s: (was: 2.0.x)
(was: 2.1.x)
2.2.0 beta 1
2.0.15
2.1.6
> Overload SecondaryIndex#indexes to accept the column definition
> ---------------------------------------------------------------
>
> Key: CASSANDRA-9314
> URL: https://issues.apache.org/jira/browse/CASSANDRA-9314
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Sergio Bossa
> Assignee: Sergio Bossa
> Fix For: 2.2.0 beta 1, 2.0.15, 2.1.6
>
> Attachments: 2.0-CASSANDRA-9314.txt
>
>
> In order to avoid rebuilding indexes when no columns are actually indexed (originally implemented in CASSANDRA-9196), CASSANDRA-9281 hinted at overriding {{SecondaryIndex#getColumnDefs}} to "filter out" indexes that shouldn't be rebuilt; unfortunately, I believe this would be quite wrong for safety reasons, as it could have side effects on other methods relying on {{getColumnDefs}} returning a stable set, and API consistency ones, as it would be confusing to have {{getColumnDefs}} behave differently based on runtime conditions, possibly ignoring {{addColumnDef}} calls.
> I believe {{SecondaryIndex#indexes}} is still the best candidate to act as a filter predicate, and I propose to overload it to accept a column definition for all those cases (i.e. index rebuilding) missing a full cell name.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)