Skip to content

chore: test hibernate against java 17+#1783

Open
karenc-bq wants to merge 1 commit intomainfrom
test/hibernate
Open

chore: test hibernate against java 17+#1783
karenc-bq wants to merge 1 commit intomainfrom
test/hibernate

Conversation

@karenc-bq
Copy link
Contributor

@karenc-bq karenc-bq commented Mar 19, 2026

Summary

Update Hibernate version to 7.3.0 and run Hibernate ORM tests against Java versions 17, 21, 22 and 24.
Update sample code to use the latest Spring and Hibernate versions.

Description

This project is built with Java 8, but the latest stable Hibernate version requires at least Java 17.
This PR creates a separate Java 17 source set for the Hibernate integration tests.
This PR also updates settings.build.kts to only compile the Spring and Hibernate sample code if the environment is using at least Java 17.

To run the tests with Java versions 17, 21, 22, 24, we had to override two additional test files:

  • settings.gradle = the minimum JDK version required to compile the project was set to 25. We had to override it to 17.
  • gradle.properties = the original derby version was 10.17.1.0, which requires at least Java 19, causing tests to fail against Java 17.

Additional Reviewers

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@karenc-bq karenc-bq force-pushed the test/hibernate branch 14 times, most recently from 5741bfb to 16d3f05 Compare March 20, 2026 17:45
if (jvm == TargetJvm.OPENJDK24 && config.noOpenJdk24) {
continue;
}
if (jvm != TargetJvm.OPENJDK24 && config.testHibernateOnly) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to change it to Java17 instead or removing this entire condition?

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.

2 participants