NobiGo commented on code in PR #4401:
URL: https://github.com/apache/calcite/pull/4401#discussion_r2108865870
##########
core/src/test/java/org/apache/calcite/test/TableInRootSchemaTest.java:
##########
@@ -210,11 +209,27 @@ protected TableWithNullableRowToplevel() {
Statement statement = calciteConnection.createStatement();
// Without the fix to this issue the Validator crashes with an
AssertionFailure:
// java.lang.RuntimeException: java.lang.AssertionError:
- // Conversion to relational algebra failed to preserve datatypes:
- // validated type:
+ // Conversion to relational algebra failed to preserve datatypes
ResultSet resultSet = statement.executeQuery("SELECT T.P.K, T.Q.S,
T.Q.S.L, T.Q.S.M FROM T");
resultSet.close();
statement.close();
connection.close();
}
+
+ /** Test case for
+ * <a
href="https://issues.apache.org/jira/browse/CALCITE-7043";>[CALCITE-7043]
+ * Type inferred for SqlItemOperator has incorrect nullability</a>. */
+ @Test void testNullableRowTopLevel2() throws Exception {
Review Comment:
Can we add test cases in `SqlOperatorTest#testItemOp`?
--
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]
