asolimando commented on a change in pull request #2553:
URL: https://github.com/apache/calcite/pull/2553#discussion_r725610316
##########
File path: core/src/test/java/org/apache/calcite/test/JdbcTest.java
##########
@@ -7742,6 +7742,17 @@ private void checkGetTimestamp(Connection con) throws
SQLException {
assertThat.query(query).returns("EXPR$0=4200000000\n");
}
+ /**
+ * Test case for
+ * <a
href="https://issues.apache.org/jira/browse/CALCITE-4811";>[CALCITE-4811]
+ * Coalesce(null, row) fails with NPE</a>.
+ */
+ @Test public void testCoalesceNullAndRow() {
+ CalciteAssert.that()
+ .query("SELECT COALESCE(NULL, ROW(1))")
Review comment:
Can you please add `CalciteAssert.isType` check as well here?
--
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]
