apache / apache/kyuubi

[Improvement] The connection to the Trino engine should be tested prior to caching

Open
#4,580 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
2.4k
Forks
1k
PR merge metrics
No merged PRs in 30d

Description

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

### Search before asking

- [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues.

### What would you like to be improved?

When connecting to the Trino engine, the kyuubi.session.engine.trino.connection.url filled in is not tested before being cached. If the connection.url is incorrect and the specified share level meets the conditions for reuse, Kyuubi will continue to reuse the incorrect connection.url, and it will not be possible to correct the connection URL by modifying the connection string. In such cases, the Kyuubi server must be restarted

### How should we improve?

Test kyuubi.session.engine.trino.connection.url and throw exception directly if the connection cannot be established to avoid reusing invalid connection.

### Are you willing to submit PR?

- [ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
- [ ] No. I cannot submit a PR at this time.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.