Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DO NOT COMMIT! #1195

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

DO NOT COMMIT! #1195

wants to merge 25 commits into from

Conversation

ekaterinadimitrova2
Copy link
Collaborator

WIP branch for testing with JDK 22

ekaterinadimitrova2 and others added 21 commits July 18, 2024 16:45
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
…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?)
…ing objects

patch by Ekaterina Dimitrova, reviewed by Benjamin Lerer for CASSANDRA-18708
…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
@ekaterinadimitrova2 ekaterinadimitrova2 marked this pull request as draft July 18, 2024 21:13
@ekaterinadimitrova2 ekaterinadimitrova2 self-assigned this Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants