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

Fix listing Glue tables with null type #24859

Merged
merged 4 commits into from
Feb 4, 2025

Conversation

pajaks
Copy link
Member

@pajaks pajaks commented Jan 31, 2025

Description

Fix listing Glue tables when table type is null.
Fixes: #24834

Release notes

(x) Release notes are required, with the following suggested text:

## Hive
* Fix NullPointerException when listing tables on Glue. ({issue}`24834`)

@cla-bot cla-bot bot added the cla-signed label Jan 31, 2025
@github-actions github-actions bot added the hive Hive connector label Jan 31, 2025
@pajaks pajaks requested review from ebyhr and wendigo January 31, 2025 13:03
@ebyhr
Copy link
Member

ebyhr commented Jan 31, 2025

/test-with-secrets sha=fa2bec0595ea9ee4a69635845635a5af9bd8afa0

Copy link

The CI workflow run with tests that require additional secrets has been started: https://github.com/trinodb/trino/actions/runs/13073389655

@pajaks pajaks force-pushed the pajaks/listing_glue_null branch from fa2bec0 to c5c2795 Compare February 3, 2025 11:29
@pajaks pajaks requested a review from ebyhr February 3, 2025 11:32
@ebyhr ebyhr force-pushed the pajaks/listing_glue_null branch from c5c2795 to 72126e2 Compare February 4, 2025 01:22
@ebyhr
Copy link
Member

ebyhr commented Feb 4, 2025

/test-with-secrets sha=72126e28d2c4e76d061b871bd84452161d6d7b0b

Copy link

github-actions bot commented Feb 4, 2025

The CI workflow run with tests that require additional secrets finished as failure: https://github.com/trinodb/trino/actions/runs/13126351102

@ebyhr ebyhr merged commit b1d2302 into trinodb:master Feb 4, 2025
97 of 98 checks passed
@github-actions github-actions bot added this to the 470 milestone Feb 4, 2025
@mosabua
Copy link
Member

mosabua commented Feb 4, 2025

Is this really only for the Hive connector @ebyhr @pajaks ?

@pajaks
Copy link
Member Author

pajaks commented Feb 5, 2025

Is this really only for the Hive connector @ebyhr @pajaks ?

I used similar classification as in similar PR #23253

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector
Development

Successfully merging this pull request may close these issues.

Trino NullPointerException when discover glue's metastore tables
3 participants