Support Charset in Quarkus Config - Use it in Hibernate

[geoand]

Relates to: #9542

Also set the default charset for Hibernate to UTF-8

[gsmet]

Added a few comments. But my main issue is that it's already implemented.

[geoand]

@gsmet given that indeed it's already implement (:facepalm:), should I just change the PR to add the converter and set UTF-8 as the default?

[Sanne]

+1 to just default to UTF-8.

But also, should we just enforce it rather than expose the configuration property? We're still trying hard to expose esclusively what can't be avoided.

If there's a real need for people to choose a charsets which is different than UTF-8, maybe we should have a "Quarkus global resource handlng" property which applies consistently to all resources handled by Quarkus? I mean: across all extensions rather than having an Hibernate specific property.

[geoand]

For now the PR was updated to just add the converter for Charset and set the default value for Hibernate to UTF-8

[geoand]

Good idea about the global UTF-8 handling. But sounds like something for a follow up PR?

[gsmet]

@Sanne we already decided to expose it as it was exposed since the very beginning (and I only exposed those that were in your list).