[Improvement] The connection to the Trino engine should be tested prior to caching
- 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
Assessment
This issue has not been assessed yet.