CNDB-11144: Support compilation and testing with j22 (by Ekaterina and co) #1323
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
patch by Ekaterina Dimitrova; reviewed by Michaem Semb Wever for CASSANDRA-17224
patch by Ekaterina Dimitrova; reviewed by Andres de la Pena and Michael Semb Wever for CASSANDRA-18267
…ld.xml, same settings as Cassandra 5.0
…roperties `java.default` and `java.supported`. ant generate-idea-files now support JDK 8, JDK 11 and JDK 22. To add support of another JDK the java-jvmargs property must be set for the JDK in question (see how it's done in build.xml for Java 11 and 22) Other minor, but notable changes are: - test jvmargs are now added to idea run configurations - .idea dir and project iml file are first removed and then recreated during `ant generate-idea-files` Based on what was done in CASSANDRA-18467, CASSANDRA-18179, CASSANDRA-18258 for 17 plus additional stuff for 21 Co-authored-by: Ekaterina Dimitrova<[email protected]> Co-authored-by: Mick Semb Wever <[email protected]> Co-authored-by: Jakub Zytka <[email protected]>
…jvm options, to match the order in the other files/places There is a chance this fixes some tests (or breakes more?)
…ASSANDRA-18723 but I figured I want to stay more conservative for this branch. This bump requires more testing than only the unit testing. Also, it looks like that some netty defaults have changed before and we agreed with that. (In Cassandra we added io.netty.allocator.useCacheForAllThreads=true, io.netty.allocator.maxOrder=11, this was not done here)
patch by Stefan Miklosovic; reviewed by Andrés de la Peña for CASSANDRA-18152 Fix for org.apache.cassandra.distributed.test.FullRepairCoordinatorFastTest
…lassCastException on JDK17 Fix ClassCastException from jdk GaloisCounterMode when using JDK17 provider Add javac exports to build.xml for java.base/jdk.internal.ref and java.base/sun.nio.ch Replace checktestnameshelper ant task in build.xml with _check-test-names java target to be able to pass jvm args to TestNameCheckTask patch by Dan Janieks; revewed by Ekaterina Dimitrova and Andres de la Pena for CASSANDRA-18180
…dit logger and vectorizedMismatch
…l want to check newer version for JDK22 specifically
This upgrade also fixes issues with PhantomReferences and the test problems from CASSANDRA-17884 anad CASSANDRA-16304 patch by Benjamin Lerer; reviewed by Ekaterina Dimitrova for CASSANDRA-18239
…revision and work if we decide to do it here; the cherry-pick was quick for testing without spending too much time on details
|
FYI THE PATCH IS ONLY FOR EXPERIMENTATION PURPOSES AT THE MOMENT! NO REVIEWS OR MERGE! If anyone wants to know details - please ping me or @jacek-lewandowski |
|
Pushed one new jamm-related commit. |
|
Remove --add-opens java.base/java.nio=ALL-UNNAMED which is not needed anymore patch by Ekaterina Dimitrova; reviewed by Jacek Lewandowki for CASSANDRA-17909
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DON'T CONSIDER THIS AS DONE
DON'T BE TEMPTED TO MERGE IT