Skip to content

Remove obsolete dialect tests and update dialect list and Regenerate metadata for Hibernate 6.6.3.Final #643

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

erguiti-imane
Copy link
Member

Remove obsolete dialect tests and update dialect list for Hibernate 6.6.3.Final

  • Removed testDeprecatedHibernateDialects() method as the tested dialects (e.g., MariaDB106Dialect, MySQL8Dialect, SQLServer2012Dialect) were removed in Hibernate 6 and the test fails.
  • Removed PostgresPlusDialect from provideHibernateDialects(), as it is no longer present in Hibernate 6.
  • Regenerated metadata

What does this PR do?

Code sections where the PR accesses files, network, docker or some external service

  • (example link to code section)

Checklist before merging

….6.3.Final

- Removed testDeprecatedHibernateDialects() method as the tested dialects
  (e.g., MariaDB106Dialect, MySQL8Dialect, SQLServer2012Dialect) were removed
  in Hibernate 6 and the test fails.
- Removed PostgresPlusDialect from provideHibernateDialects(), as it is no longer
  present in Hibernate 6.
- Regenerated metadata
@erguiti-imane erguiti-imane requested a review from a team as a code owner June 16, 2025 16:09
@erguiti-imane erguiti-imane requested a review from vjovanov June 16, 2025 16:09
@erguiti-imane erguiti-imane force-pushed the orm-hibernate-core/fix-6.6.3 branch from feec951 to 0853551 Compare June 17, 2025 08:46
@erguiti-imane erguiti-imane force-pushed the orm-hibernate-core/fix-6.6.3 branch from 0853551 to f047dc4 Compare June 17, 2025 08:52
@@ -0,0 +1,4 @@
[
Copy link
Member

Choose a reason for hiding this comment

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

@@ -0,0 +1,15 @@
[
Copy link
Member

Choose a reason for hiding this comment

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

Once all necessary metadata is copied to metadata/org.hibernate.orm/hibernate-core/6.6.3.Final please remove generated-metadata directory, as we only care about what is under root level metadata directory

@dnestoro
Copy link
Member

@sdeleuze can you please take a look on this PR as you were the initial author of Hibernate tests

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants