Skip to content

Upgrade to Hibernate 6 #31663

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

Closed
wants to merge 2 commits into from

Conversation

odrotbohm
Copy link
Member

Initial draft of a Hibernate 6 upgrade for Spring Boot. The first commit upgrades to Spring Data JPA and Envers versions that use Hibernate 6 already. This commit can be removed from the PR once spring-projects/spring-data-jpa#2423 is merged. We didn't merge that yet, as it is likely to break Boot unless it also upgrades to H6.

The upgrade removes support for the USE_NEW_ID_GENERATOR_MAPPINGS settings as H6 had that removed, unfortunately without prior deprecation in 5.6. Does it make sense to deprecate the Boot setting triggering that setting in its 2.7 branch then?

I also had to adapt the sequence definitions for Liquibase and Flyway, as the names H6 expects are different (entity specific) now.

odrotbohm added 2 commits July 9, 2022 15:49
Also fix Reactor Netty HTTP to 1.1 M2 as the Data update seems to leak snapshots of the former into the build effectively breaking it as the Reactor snapshots introduce deprecations Boot compiles against.
Upgraded Hibernate artifact coordinates and version to Hibernate 6. A couple of previously referred to modules are not available in H6 anymore. They likely need review.

Removed unsupported properties (AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS has been removed entirely) from the configurable properties (deprecate in 2.7.x?).
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 9, 2022
@wilkinsona wilkinsona self-assigned this Jul 11, 2022
@wilkinsona
Copy link
Member

Whoops. I forgot that we had a PR for the changes. They've been committed on main as part of #31674.

@wilkinsona wilkinsona closed this Jul 12, 2022
@wilkinsona wilkinsona added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants