hibernate.* properties in Hibernate-ORM extension

[neon-dev]

Description
I am missing a way to configure hibernate.event.merge.entity_copy_observer (see) from application.properties.
Using a persistence.xml I can set all kinds of properties but as I read, in terms of Quarkus it's "the old way" of doing things and not encouraged.

Implementation ideas
If I understand correctly, quarkus.hibernate-orm.* properties are global. So where I could set hibernate.show_sql per persistence unit before, I can now only set quarkus.hibernate-orm.log.sql for all datasources using Hibernate? It might be useful to set these properties per datasource. Also I think it'd be better to implement such properties in a pass-through way, so it's no question anymore whether Hibernate property X can be specified via application.properties.

[quarkusbot]

/cc @gsmet, @Sanne

[famod]

I have a similar problem: I need to register a custom StatementInspector.
See also: https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Hibernate.20StatementInspector

[Sanne]

+1 to have way to observe merge events.

But as @famod linked to, I believe that we shouldnt' expose the configurfation properties but rather expose a simple way to register listeners and customized components linked to the persistence unit.

I definitely don't want to "forward" all hibernate.* properties; we're restricting them for various good reasons.

[neon-dev]

we're restricting them for various good reasons

Could you please elaborate? It works with a persistence.xml, so why artificially limit configuration properties for datasources defined purely in application.properties?
Also, is there currently a way to set hibernate.event.merge.entity_copy_observer=allow without a dedicated persistence.xml?

[frederikb]

May I respectfully suggest considering the addition of the hibernate.use_sql_comments option to complete the SQL logging trifecta? It is a helpful debugging tool.

Explanation/Example: https://dzone.com/articles/hibernate-debugging-where-does
Reference: https://docs.jboss.org/hibernate/orm/5.4/userguide/html_single/Hibernate_User_Guide.html#_sql_statement_logging

[goafabric]

I also have the Requirement to register a StatementInspector.
Is there any progress for this issue ?

[yrodiere]

Duplicates #19129

This was fixed in Quarkus 2.11.0.CR1, see #26128