Reactive - Support multi hosts connection to Postgres

[mcanzerini]

Description

I need to connect to multiple postgres hosts with hibernate-reactive.

With the classic jdbc driver, we can define this property to connect to our HA postgres instance:

quarkus.datasource.jdbc.url=jdbc:postgresql://my.host-1.com,my.host-2.com,my.host-3.com:5432/myDB

But here I saw that the vert.x PgClient does not support multi host connections directly in connection URI.

I created an issue in vertx-sql-client here and a developer said me that it would be already possible by using the PgConnectOptions and a PgPool.

Can you support connection pool to multi hosts through your implementation ?

[quarkus-bot]

/cc @tsegismont, @vietj

[tsegismont]

@geoand do you know if it is possible to customize connect options in user code?

@mcanzerini as a workaroud, you create the PgPool directly in your application (with your own config options)

[geoand]

@geoand do you know if it is possible to customize connect options in user code?

If you are looking for a way to customize the PgConnectOptions that Quarkus creates, then no. But I do think it would be something useful.
If we do decide to add something like that, we'll need to do it for all reactive drivers (and probably the non-reactive ones as well).

Or maybe it makes sense for users to customize the pool instead? Let me know what you think and I can provide an initial PR showing how this can be done.

[tsegismont]

Are there some extensions which do something similar and which I could inspire from?

[geoand]

None of the datasource extensions do this IIRC.

I'll open a draft tomorrow to show you what I have in mind (which is in line with the way other extensions like Jackson for example handle such cases)

[tsegismont]

Cool, thanks

[geoand]

you create the PgPool directly in your application (with your own config options)

If you do this, make sure to make the PgPool a bean in order to override the Quarkus provided bean.

[geoand]

@tsegismont #29397 is what I have in mind (cc @gsmet for awareness as well).
If we agree on that approach, I'll do the same thing for the other reactive drivers as well.

[mcanzerini]

Hi guys, any update on this ?
Thanks !

[geoand]

Hopefully soon :)

[mcanzerini]

Great, thanks for your time 🙏