-
Notifications
You must be signed in to change notification settings - Fork 3k
ISSUE-32936 | Allow plugging in Hibernate ORM's TypeContributor and FunctionContributor #51323
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
base: main
Are you sure you want to change the base?
Conversation
00f29ea to
2bc8557
Compare
|
/cc @gsmet (hibernate-orm) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note you'll need to update docs as well, to mention the new component types in that list:
quarkus/docs/src/main/asciidoc/hibernate-orm.adoc
Lines 543 to 561 in 206858a
| ==== Plugging in other custom components | |
| The Quarkus extension for Hibernate ORM will automatically | |
| inject components annotated with `@PersistenceUnitExtension` into Hibernate Search. | |
| The annotation can optionally target a specific persistence unit with `@PersistenceUnitExtension(name = "nameOfYourPU")`. | |
| This feature is available for the following component types: | |
| `org.hibernate.Interceptor`:: | |
| See <<interceptors>>. | |
| `org.hibernate.resource.jdbc.spi.StatementInspector`:: | |
| See <<statement_inspectors>>. | |
| `org.hibernate.type.format.FormatMapper`:: | |
| See <<json_xml_serialization_deserialization>>. | |
| `io.quarkus.hibernate.orm.runtime.tenant.TenantResolver`:: | |
| See <<multitenancy>>. | |
| `io.quarkus.hibernate.orm.runtime.tenant.TenantConnectionResolver`:: | |
| See <<programmatically-resolving-tenants-connections>>. |
...orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java
Outdated
Show resolved
Hide resolved
...orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java
Outdated
Show resolved
Hide resolved
| private void applyMetadataBuilderContributor() { | ||
| Object metadataBuilderContributorSetting = buildTimeSettings | ||
| .get(JpaSettings.METADATA_BUILDER_CONTRIBUTOR); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remember to deprecate the metadata build contributor, now that you're introducing an alternative.
A @Deprecated annotation + @deprecated javadoc tag there would do the trick:
Line 141 in 02cb12f
| Optional<@WithConverter(TrimmedStringConverter.class) String> metadataBuilderContributor(); |
…tributions" This reverts commit 00f29ea.
This PR fixes 32936