Introduce support for hibernate.type.preferred_*_jdbc_type properties

[NickBeginner]

Add Quarkus properties to map the following hibernate properties:

• hibernate.type.preferred_duration_jdbc_type : quarkus.hibernate-orm.mapping.duration.preferred_jdbc_type

• hibernate.type.preferred_instant_jdbc_type : quarkus.hibernate-orm.mapping.duration.preferred_instant_jdbc_type

• hibernate.type.preferred_boolean_jdbc_type : quarkus.hibernate-orm.mapping.preferred.boolean_jdbc_type

• hibernate.type.preferred_uuid_jdbc_type : quarkus.hibernate-orm.mapping.preferred.uuid_jdbc_type

• Fixes Support properties hibernate.type.preferred_*_jdbc_type #33328

[quarkus-bot]

/cc @gsmet (hibernate-orm)

[yrodiere]

Thank you. I added a few comments below.

[NickBeginner]

@yrodiere thanks saw it, is still in draft so I will take them in account while i will be writing the remaining part!

Thanks a lot 🙏🏻

[NickBeginner]

@yrodiere First issue so i don't know if i missed something :)

[yrodiere]

Thanks, but the config group stuff and property naming is really off. See my comment in the conversation above for a suggestion on how to do it :)

[yrodiere]

Thanks. The properties shouldn't be in application.properties in this case (see below). And ideally it would be great to have an integration test, but I'll let you see if you have time for that :)

[github-actions]

🎊 PR Preview ead1e66 has been successfully built and deployed to https://quarkus-pr-main-50454-preview.surge.sh/version/main/guides/

• Images of blog posts older than 3 months are not available.
• Newsletters older than 3 months are not available.

[yrodiere]

Thanks! Looking great, though I suspect native tests will fail. See below.

I'll launch CI to be sure...

[yrodiere]

Thanks for the updates @NickBeginner . I made a few changes to fix the tests (see below), squashed, and rebased on main.

I think it's ready to merge if CI passes. Let's see!

[quarkus-bot]

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit f64d9d7.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit f64d9d7.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

[yrodiere]

It took a few tries, but it finally works :) Merged, thank you @NickBeginner!

[NickBeginner]

@yrodiere great!!!! Thanks to you for the help on understanding how to assert the last two things,I didn't know how to proceed.

Thanks a lot 🙏🏻