caicancai commented on code in PR #3905:
URL: https://github.com/apache/calcite/pull/3905#discussion_r1712505415
##########
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##########
@@ -2424,6 +2424,14 @@ private static void checkConcatFunc(SqlOperatorFixture
f) {
f.checkFails("^concat()^", INVALID_ARGUMENTS_NUMBER, false);
}
+ /** Test case for <a
href="https://issues.apache.org/jira/browse/CALCITE-6518">
+ * ClassCastException during validation when loading multiple libraries</a>.
*/
+ @Test void issue6518() {
Review Comment:
I'm not sure issue6518 is appropriate, as there doesn't seem to be a similar
test starting with that issue yet
##########
core/src/main/java/org/apache/calcite/sql/SqlKind.java:
##########
@@ -29,7 +29,7 @@
* Enumerates the possible types of {@link SqlNode}.
*
* <p>The values are immutable, canonical constants, so you can use Kinds to
- * find particular types of expressions quickly. To identity a call to a common
+ * find particular types of expressions quickly. To identify a call to a common
Review Comment:
good catch
--
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]