Quarkus: SpringJPA: Support spring.jpa.show-sql

[aureamunoz]

Description
From a Hackaton with Quarkus suggestions and feedbacks:
User tried to set spring.jpa.show-sql was surprised it didn’t work.

Implementation ideas
My idea is create a @ConfigRoot and @ConfigGroup classes to load this property and map it to the current quarkus.hibernate-orm.log.sql. Anyway the new spring property would be named quarkus.spring.jpa.show-sql, we cannot escape to have quarkus in the name. WDYT @geoand ?

[geoand]

TBH I don't have a proper solution in my head, so I would suggest you go ahead and experiment and see what actually works best.

[gsmet]

I think this asks a broader question: should we support the whole Spring configuration? Because once we have added one, it's weird to not support the others.

I must admit that I'm not sold to it.

[geoand]

I am also against adding support because it then open doors other types of Java config for Spring. Would however explore if we can detect some specific configuration options that the user might have added and perhaps propose the proper Quarkus alternatives?

[gsmet]

Would however explore if we can detect some specific configuration options that the user might have added and perhaps propose the proper Quarkus alternatives?

That would be handy and probably relatively easy to implement.

[Sanne]

I see #6291 was merged; should we close this one?

[geoand]

Definitely, thanks for catching it @Sanne