Skip to content

Conversation

@ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Nov 24, 2025

@ppalaga
Copy link
Contributor Author

ppalaga commented Nov 24, 2025

The CI failure does not seem to be related to this PR.
@ffang or anybody else, could you please restart the failing job?

@coheigea
Copy link
Contributor

@ppalaga I normally just rebase PRs to re-start the jobs. One thing I noticed locally with jDK17 is that Derby as a test dependency in cxf-rt-ws-rm needs JDK 21, so we'll need to update surefire for that module to only run certain tests depending on the JDK version

@reta
Copy link
Member

reta commented Nov 24, 2025

@ppalaga the change to use JDK-21 is intentional, see please [1] for discussion here, the original plan for 4.2.x (main) was to be on JDK-21 baseline.

[1] https://lists.apache.org/thread/8q8jrgdkgqys4fhgwyxsj2hjt171m114

@reta
Copy link
Member

reta commented Nov 24, 2025

Following up on the possible options, I think we could cleanly support 17 and 21, @ppalaga do you mind if I push a change to your pull request? (or I could open a separate one, but you did a good chunk of work here, thank you)

@reta
Copy link
Member

reta commented Nov 25, 2025

Fixed by #2754

@ppalaga
Copy link
Contributor Author

ppalaga commented Nov 26, 2025

@reta I apologize, I got distracted by other tasks. Thanks a lot, #2754 works great, our jobs testing against main are all green again!

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.

3 participants