Skip to content

Update trino driver to 438#2372

Merged
alexander-skoblikov merged 5 commits intodbeaver:develfrom
alaturqua:patch-1
Mar 28, 2024
Merged

Update trino driver to 438#2372
alexander-skoblikov merged 5 commits intodbeaver:develfrom
alaturqua:patch-1

Conversation

@alaturqua
Copy link
Copy Markdown
Contributor

@alaturqua alaturqua commented Feb 8, 2024

Summary
This PR updates the Trino dependency from its current version to version 438. The primary goal of this update is to address and resolve compatibility issues that have arisen with Trino installations, specifically for versions 429 and above.

Background
We have identified that the version of the Trino driver currently utilized within cloudbeaver project does not function optimally with Trino version 429 and later. This has led to users experiencing problems due to the incompatibilities between the older driver version and newer versions of Trino installations. The specifics of these issues include runtime errors and failures in connectivity, impacting the overall user experience and the reliability of our project when used in conjunction with these versions of Trino.

@LonwoLonwo
Copy link
Copy Markdown
Member

Hello @alaturqua

Sorry for the late response. Could you please add more info to the PR description? Reasons to update Trino driver?

@alaturqua
Copy link
Copy Markdown
Contributor Author

Hello @LonwoLonwo,

I initiated the PR to update the Trino dependency to version 438 to address compatibility issues with Trino installations newer than current dependency version 422, specifically version 429 and above. Users of our project who are on Trino 429 or later have encountered problems due to incompatibilities with the older Trino driver version we currently utilize. Although I don't have the exact error messages handy at this moment, the motivation for this update is to ensure broader compatibility and stability with recent Trino versions, enhancing user experience and avoiding potential runtime issues.

@LonwoLonwo
Copy link
Copy Markdown
Member

Thanks for the answer @alaturqua and for the clear description.

@LonwoLonwo LonwoLonwo requested a review from yagudin10 March 4, 2024 12:41
@LonwoLonwo LonwoLonwo added the external For external PRs label Mar 5, 2024
@LonwoLonwo LonwoLonwo linked an issue Mar 6, 2024 that may be closed by this pull request
@alexander-skoblikov alexander-skoblikov merged commit 77f531d into dbeaver:devel Mar 28, 2024
@LonwoLonwo
Copy link
Copy Markdown
Member

Thank you for your contribution and your patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external For external PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Trino driver.

4 participants