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)