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

Add initial support for hibernate-orm:hibernate-core:6.5.0.Final #479

Merged
merged 6 commits into from
May 12, 2024

Conversation

christophstrobl
Copy link
Contributor

@christophstrobl christophstrobl commented Apr 12, 2024

What does this PR do?

Move over and adapt tests from hibernate 6.2 to generate metadata via generate-metadata.sh script.

Fixed compilation issues and removed no longer supported dialects.
Updated metadata generation to include Logger setup.
Updated metadata filter to ignore hibernate internal code generation facilities.

Checklist before merging

  • I have considered including reachability metadata directly in the library or the framework (see our contributing document).
    A: Given the metadata already resides in this repository I think it's fine to adapt for newer versions of the target library.
  • I am the original author of all content provided in the pull request, and I did not copy the content from any other source (see the tests section from our contributing document).
    A: Actually I copied existing tests form this repository to a new directory and modified them so they run with the updated version of the library.
  • For all tests where I am not the sole author, I have added a comment that proves I may publish them under the specified license (see the tests section from our contributing document).
    A: Well those are copies of existing tests in the very same repository & license.
  • I have properly formatted metadata files (see our formatting guide)
  • I have added thorough tests (see the tests section from our contributing document)
  • I have filled all places where my pull request accesses files, network, docker, or any other external service (see the section above)

@christophstrobl christophstrobl requested a review from a team as a code owner April 12, 2024 09:13
@christophstrobl christophstrobl requested a review from olpaw April 12, 2024 09:13
@christophstrobl christophstrobl marked this pull request as draft April 12, 2024 09:25
@christophstrobl christophstrobl force-pushed the hibernate-orm-core/6.5.0.CR1 branch from 3e7e7ae to ee8d745 Compare April 15, 2024 07:42
@christophstrobl christophstrobl marked this pull request as ready for review April 15, 2024 07:51
@christophstrobl
Copy link
Contributor Author

Hibernate 6.5 has been released (hibernate-orm:6.5.0.Final )- will update the PR with new version.

Move over and adapt tests from hibernate 6.2 to generate metadata via generate-metadata script.

Fixed compilation issues and removed no longer supported dialects.
Updated metadata generation to include Logger setup.
Updated metadata filter to ignore hibernate internal code generation facilities.
the bytecode provider property is no longer supported so we need to bypass the default implementation to avoid failures with bytebuddy.
@christophstrobl christophstrobl force-pushed the hibernate-orm-core/6.5.0.CR1 branch from ee8d745 to 2c24c98 Compare April 26, 2024 12:25
@christophstrobl christophstrobl changed the title Add initial support for hibernate-orm:hibernate-core:6.5.0.CR1 Add initial support for hibernate-orm:hibernate-core:6.5.0.Final Apr 26, 2024
@@ -0,0 +1,2 @@
hibernate.bytecode.provider=none
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this file be deleted since these properties are not read in this version of hibernate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, sorry - overlooked that one

@sdeleuze sdeleuze self-requested a review April 29, 2024 12:33
@sdeleuze
Copy link
Collaborator

Tested with Petclinic, Spring Boot 3.3.0-RC1 and Hibernate 6.5.0, works as expected.

@fniephaus fniephaus merged commit e6fe400 into oracle:master May 12, 2024
13 checks passed
@fniephaus fniephaus self-assigned this May 12, 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