yashmayya commented on code in PR #18195:
URL: https://github.com/apache/pinot/pull/18195#discussion_r3114227961
##########
pinot-common/src/main/java/org/apache/pinot/common/evaluator/InbuiltFunctionEvaluator.java:
##########
@@ -360,6 +498,12 @@ public Object[] execute(Object[] values) {
return _value;
}
+ @Nullable
+ @Override
+ public ColumnDataType getResultType() {
Review Comment:
Good catch, thanks! Addressed in 4eae49f48a — `ArrayConstantExecutionNode`
now infers the array `ColumnDataType` from the literal elements so `mvCol =
ARRAY['a','b']` dispatches to `stringArrayEquals` and the other typed array
overloads. Mixed element types, nested arrays, and types without an array
variant still return null and fall back to arity-based lookup. Added tests for
both the working case and the mixed-type fallback.
--
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]
