bchapuis commented on code in PR #3668:
URL: https://github.com/apache/calcite/pull/3668#discussion_r2243855299


##########
core/src/main/java/org/apache/calcite/adapter/jdbc/JdbcToEnumerableConverter.java:
##########
@@ -323,6 +323,14 @@ private static void generateGet(EnumerableRelImplementor 
implementor,
               java.sql.Array.class);
       source = Expressions.call(BuiltInMethod.JDBC_ARRAY_TO_LIST.method, x);
       break;
+    case GEOMETRY:

Review Comment:
   My integration tests are located here: 
https://github.com/bchapuis/calcite-postgis-tests
   
   I looked into how Calcite uses Testcontainers, but it seems the support is 
currently limited to a few Redis tests. The JDBC tests don’t appear to leverage 
Testcontainers yet, which is why I set up external integration tests. Did I 
overlook anything?



-- 
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]

Reply via email to