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



##########
File path: src/java/org/apache/cassandra/index/Index.java
##########
@@ -364,6 +382,23 @@ default SSTableFlushObserver getFlushObserver(Descriptor 
descriptor, OperationTy
      */
     public AbstractType<?> customExpressionValueType();
 
+    /**
+     * If the index supports custom search expressions using the
+     * {@code SELECT * FROM table WHERE expr(index_name, expression)} syntax, 
this method should return a new
+     * {@link RowFilter.CustomExpression} for the specified expression value. 
Index implementations may provide their
+     * own implementations using method {@link 
RowFilter.CustomExpression#isSatisfiedBy(TableMetadata, DecoratedKey, Row)}
+     * to filter reconciled rows in the coordinator. Otherwise, the default 
implementation will accept all rows.
+     * See DB-2185 and DSP-16537 for further details.
+     *
+     * @param metadata the indexed table metadata
+     * @param value the custom expression value
+     * @return a custom index expression for the specified value
+     */
+    default RowFilter.CustomExpression customExpressionFor(TableMetadata 
metadata, ByteBuffer value)

Review comment:
       removed




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to