CASSJAVA-115: Java 25 support - #2085
Conversation
| <!-- allow deep reflection for mockito when using JDK 17+, see https://stackoverflow.com/questions/70993863/mockito-can-not-mock-random-in-java-17 --> | ||
| <mockitoopens.argline>--add-opens=java.base/jdk.internal.util.random=ALL-UNNAMED</mockitoopens.argline> | ||
| </properties> | ||
| </profile> |
There was a problem hiding this comment.
I did away with the "test-jdk-X" blocks as part of the work on CASSJAVA-135. The blocks for all support JDK versions (Java17 and Java21 in that example) were identical so there wasn't really any need to keep it in place. These args match what will be the defaults once that PR is merged so this entire profile can just be deleted.
|
Looks about like what I'd expect @SiyaoIsHiding. I'm going to kick off a Jenkins run just to make sure nothing odd happens but if that's clean I'll argue we should just merge this. We'll need to resolve a few things against changes coming in CASSJAVA-135 but I can do that there just as easily as you can do it here... and I strongly suspect we'll be ready to merge this change before we're ready to merge CASSJAVA-135. |
2 participants
Add Java 25 to DataStax CI. Adding Java 25 to ASF CI will be in another PR.
Attention needed on: