You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sam Tunnicliffe (JIRA)" <ji...@apache.org> on 2016/01/08 18:55:39 UTC

[jira] [Commented] (CASSANDRA-10924) Pass base table's metadata to Index.validateOptions

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

Sam Tunnicliffe commented on CASSANDRA-10924:
---------------------------------------------

Hmm, yes I can see how that may make validation tricky for some custom implementations. The problem is with changing the method signature in a non-major version. I know it will complicate {{IndexMetadata}} somewhat, but could we extend the check to look for both signatures and call whichever is defined? 


> Pass base table's metadata to Index.validateOptions
> ---------------------------------------------------
>
>                 Key: CASSANDRA-10924
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10924
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: CQL, Local Write-Read Paths
>            Reporter: Andrés de la Peña
>            Assignee: Andrés de la Peña
>            Priority: Minor
>              Labels: 2i, index, validation
>             Fix For: 3.0.x, 3.x
>
>         Attachments: CASSANDRA-10924-v0.diff
>
>
> Some custom index implementations require the base table's metadata to validate their creation options. For example, the options of these implementations can contain information about which base table's columns are going to be indexed and how, so the implementation needs to know the existence and the type of the columns to be indexed to properly validate.
> The attached patch proposes to add base table's {{CFMetaData}} to Index' optional static method to validate the custom index options:
> {{public static Map<String, String> validateOptions(CFMetaData cfm, Map<String, String> options);}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)