[GitHub] [cassandra] jasonstack commented on a change in pull request #844: CASSANDRA-16331 Extend the exclusion of replica filtering protection to other indices instead of just SASI

[GitBox <gi...@apache.org>]

jasonstack commented on a change in pull request #844:
URL: https://github.com/apache/cassandra/pull/844#discussion_r538605610



##########
File path: src/java/org/apache/cassandra/index/Index.java
##########
@@ -426,6 +426,16 @@ public Indexer indexerFor(DecoratedKey key,
                               WriteContext ctx,
                               IndexTransaction.Type transactionType);
 
+    /**
+     * Tells whether this index supports replica fitering protection or not.
+     *
+     * @return true if this index supports replica filtering protection, false otherwise
+     */
+    default boolean supportsReplicaFilteringProtection()

Review comment:
       how about making it more generic? like `supports(IndexFeature feature)`. it will be useful for future indexes. WDYT?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org