xiangfu0 commented on code in PR #18165:
URL: https://github.com/apache/pinot/pull/18165#discussion_r3072608886


##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/function/FunctionEvaluator.java:
##########
@@ -18,27 +18,10 @@
  */
 package org.apache.pinot.segment.local.function;
 
-import java.util.List;
-import org.apache.pinot.spi.data.readers.GenericRow;
-
-
 /**
- * Interface for evaluators of transform function expressions of schema field 
specs
+ * @deprecated Use {@link 
org.apache.pinot.segment.spi.function.FunctionEvaluator}. This bridge remains 
so existing
+ *             segment-local callers continue to compile while the interface 
lives in {@code pinot-segment-spi}.
  */
-public interface FunctionEvaluator {
-
-  /**
-   * Get the arguments of the function
-   */
-  List<String> getArguments();
-
-  /**
-   * Evaluate the function on the generic row and return the result
-   */
-  Object evaluate(GenericRow genericRow);
-
-  /**
-   * Evaluates the function on the given values (same order as the arguments) 
and returns the result.
-   */
-  Object evaluate(Object[] values);
+@Deprecated
+public interface FunctionEvaluator extends 
org.apache.pinot.segment.spi.function.FunctionEvaluator {

Review Comment:
   delete this and refactor to use pinot-segment-spi module 



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