Skip to content

[jdbc-v2] Set default ports, parse multiple endpoints #2488

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

Closed
wants to merge 1 commit into from

Conversation

enqueue
Copy link
Contributor

@enqueue enqueue commented Jul 8, 2025

Summary

Default port

Client endpoint URLs need to have a port set (see Client). The JdbcConfiguration will use the default ports depending on ssl parameter / http(s) prefix.

Multiple endpoints in JDBC URL

I know that only one endpoint will currently be used, and there is an ongoing discussion in #1870 whether we want to make it the driver's responsibility to maintain liveness states, distribute calls, keep a pool of connections etc. But the current documentation explains that it is possible to give multiple endpoints. Without this PR this would lead to broken connection. With this PR it is more likely that a successful connection can be established. Alternatively "fix" documentation and throw SQLException right away when we detect multiple endpoints in JDBC URL. This would make everything easier 😄 . See #2489 which is an alternative to this PR.

Checklist

  • [ ✅ ] Unit and integration tests covering the common scenarios were added

@enqueue enqueue changed the title Set default ports, parse multiple endpoints [jdbc-v2] Set default ports, parse multiple endpoints Jul 8, 2025
@enqueue
Copy link
Contributor Author

enqueue commented Jul 10, 2025

This is not needed, since multiple endpoints should not be supported via JDBC URL.

@enqueue enqueue closed this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant