asolimando opened a new pull request #2592: URL: https://github.com/apache/calcite/pull/2592
## Summary The PR upgrades to Cassandra 4 and to the DataStax driver 4.13.0. ## Version (in)compatibilities In terms of version compatibility, there are the following news: 1. Cassandra 4 is now compatible with `Guava` versions > 25 2. Cassandra is still not compatible with `Eclipse OpenJ9` JVM: see [https://issues.apache.org/jira/browse/CASSANDRA-14883](CASSANDRA-14883) 3. Still a problem with JDK11 due to `cassandra-unit`: > CassandraAdapterTest STANDARD_OUT > 2021-10-21 16:29:48 [ForkJoinPool-1-worker-19] ERROR o.apache.cassandra.io.util.FileUtils - FATAL: Cassandra is unable to access required classes. This usually means it has been run without the aid of the standard startup scripts or the scripts have been edited. If this was intentional, and you are attempting to use Java 11+ you may need to add the --add-exports and --add-opens jvm options from either jvm11-server.options or jvm11-client.options > java.lang.IllegalAccessException: access to public member failed: jdk.internal.ref.Cleaner.clean[Ljava.lang.Object;@15b79143/invokeVirtual, from org.apache.cassandra.io.util.FileUtils (unnamed module @4518a223) > There is an open issue (with a pending PR since May 2021) providing more details: https://github.com/jsevellec/cassandra-unit/issues/294 Other useful information are in this Datastax support page: https://support.datastax.com/s/article/ERROR-FATAL-Cassandra-is-unable-to-access-required-classes-This-usually-means-it-has-been On the cassandra-unit front, it does not seem to be advancing since the project does not seem actively maintained anymore, so either we find a replacement, or we are still stuck with JDK8 for the Cassandra tests. So, given 2. and 3., Cassandra tests are now skipped for JDK11+ and on OpenJ9 JVM (this isn't new, but the incompatibility was "hidden" by the Guava version used in our CI employing OpenJ9, which was previously incompatible). ## Commits The PR has multiple commits for easing the review process, here is a summary of what each one covers: - 01f2b4dbe Skip Cassandra tests on unsupported OpenJ9 - c7b59a9fa fixed warnings (mostly nullability annotations) <-- some code cleanup and removing warnings, if unwanted it can be dropped but I think it's nice to have, especially since many nullability annotations were missing or wrong - e75333853 code update <-- the actual code changes due to the version up - 368ebb1c7 cassandra config update <-- Cassandra config changed, so the one used by our embedded Cassandra for tests had to follow as well - 94e97ae57 dependencies version up -- 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]
