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

Mike Adamson commented on CASSANDRA-19367:
------------------------------------------

The obvious place for this would be in {{IndexTermType}} where we could replace 
{{isLiteral}} and {{isVector}} with a {{getStrategy}} (or some such). The 
{{Strategy}} would then need to handle all the conditionals where the above 
methods are used.

Apart from anything this would tidy a lot of the current code paths where we 
are constantly checking for the index type.

> Refactor SAI so the selection of the index type is not scattered to multiple 
> places
> -----------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-19367
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-19367
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Feature/2i Index
>            Reporter: David Capwell
>            Assignee: David Capwell
>            Priority: Normal
>             Fix For: 5.x
>
>
> For Accord we want to write an internal index and finding plugging into SAI 
> is a bit more channeling than it could be… we need to find multiple places 
> where the SAI code “infer” the index type so it can delegate… this logic 
> should be done once and made pluggable so custom SAI indexes can be defined



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to