chucheng92 commented on code in PR #3579:
URL: https://github.com/apache/calcite/pull/3579#discussion_r1427505058
##########
core/src/main/java/org/apache/calcite/sql/type/OperandTypes.java:
##########
@@ -1225,6 +1229,64 @@ private static class MapFromEntriesOperandTypeChecker
}
}
+ /**
+ * Operand type-checking strategy for a ARRAY function, it allows empty
array.
+ *
+ * <p> The reason it overrides SameOperandTypeChecker#checkOperandTypesImpl
is that it needs
+ * to handle the scenario where row/struct type and NULL exist
simultaneously in array.
+ * This scenario need be supported, but will be rejected by the current
checkOperandTypesImpl.
Review Comment:
yes, 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]
