chucheng92 commented on code in PR #3458:
URL: https://github.com/apache/calcite/pull/3458#discussion_r1349539324
##########
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##########
@@ -1075,7 +1079,8 @@ private static RelDataType
arrayReturnType(SqlOperatorBinding opBinding) {
public static final SqlFunction ARRAY =
SqlBasicFunction.create("ARRAY",
SqlLibraryOperators::arrayReturnType,
- OperandTypes.SAME_VARIADIC);
+ OperandTypes.SAME_VARIADIC,
+ SqlFunctionCategory.SYSTEM);
Review Comment:
current the function type of ARRAY is Numeric currently. Obviously not
right. The closest type in calcite is System. it means built-in or system-in.
we can see many other functions if not belongs to timedate/string/numeric will
let it be System. more details please see ANSI sql function category:
https://www.oreilly.com/library/view/sql-in-a/9780596155322/ch04s04.html.
It looks like that the calcite System category equals built-in category.
--
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]
