Skip to content
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

[#1154] fix(trino-connector): Fix the issue with joins causing errors in PostgreSQL. #1186

Conversation

github-actions[bot]
Copy link

What changes were proposed in this pull request?

Fix the issue with joins causing errors in PostgreSQL.
Error message is "Cannot cast com.datastrato.gravitino.trino.connector.GravitinoColumnHandle to io.trino.plugin.jdbc.JdbcColumnHandle"

Why are the changes needed?

Fix: #1154

Does this PR introduce any user-facing change?

NO

How was this patch tested?

UT

… in PostgreSQL. (#1177)

### What changes were proposed in this pull request?

Fix the issue with joins causing errors in PostgreSQL.
Error message is "Cannot cast
com.datastrato.gravitino.trino.connector.GravitinoColumnHandle to
io.trino.plugin.jdbc.JdbcColumnHandle"

### Why are the changes needed?

Fix: #1154

### Does this PR introduce _any_ user-facing change?

NO

### How was this patch tested?

UT
@github-actions github-actions bot added cherry-pick need backport Issues that need to backport to another branch labels Dec 18, 2023
@jerryshao jerryshao closed this Dec 18, 2023
@jerryshao jerryshao reopened this Dec 18, 2023
Copy link
Author

Code Coverage Report

Overall Project 65.97% 🟢
Module Coverage
trino-connector 76.88% 🟢
Files
Module File Coverage
trino-connector GravitinoSplitManager.java 100% 🟢

@jerryshao jerryshao merged commit ccc767e into branch-0.3 Dec 18, 2023
4 checks passed
@jerryshao jerryshao deleted the cherry-pick-branch-0.3-20558b6e3b270e0df88eaaeb2f4e4a2ada3b39d4 branch December 21, 2023 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick need backport Issues that need to backport to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants