New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More reactive SQL client connection options #11498
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a suggestion. Happy to discuss it as I'm not really sure.
...me/src/main/java/io/quarkus/reactive/datasource/runtime/DataSourceReactiveRuntimeConfig.java
Outdated
Show resolved
Hide resolved
@machi1990 we have a doc generation issue with the duration note. We include it twice in the datasource.doc now that we have it included here too. I don't exactly know how we could fix that as these things are generated totally independently. |
64e34d5
to
39c5a9f
Compare
I will try to figure out how to fix the doc issue. |
Thanks
Le jeu. 20 août 2020 à 16:07, Guillaume Smet <notifications@github.com> a
écrit :
… I will try to figure out how to fix the doc issue.
|
This is useful when you have several config references included in the same document and you only want one note. We can probably do better but I prefer to have a quick fix to unblock this PR.
…eactive datasource config
39c5a9f
to
238e305
Compare
I added a quick (and dirty) fix for the doc issue. Let's see what CI says. |
@gsmet @tsegismont sorry only seeing this comment now. Good thing we had a fix in place :-) |
Added reconnectAttempts, reconnectInterval and idleTimeout props to reactive datasource config
Resolves #11149