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

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

v1 looks good, thanks! I've pushed branches for CI and will commit them when 
cassci is happy.

||branch||testall||dtest||
|[10924-3.0|https://github.com/beobal/cassandra/tree/10924-3.0]|[testall|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-3.0-testall]|[dtest|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-3.0-dtest]|
|[10924-3.3|https://github.com/beobal/cassandra/tree/10924-3.3]|[testall|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-3.3-testall]|[dtest|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-3.3-dtest]|
|[10924-trunk|https://github.com/beobal/cassandra/tree/10924-trunk]|[testall|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-trunk-testall]|[dtest|http://cassci.datastax.com/view/Dev/view/beobal/job/beobal-10924-trunk-dtest]|


> 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, CASSANDRA-10924-v1.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)

Reply via email to