-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add support for dropping corrupted tables to Iceberg Nessie Catalog #20360
Add support for dropping corrupted tables to Iceberg Nessie Catalog #20360
Conversation
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Emmanuel Pescosta.
|
4c5c503
to
a8f77f7
Compare
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
Please fix CI failure and submit CLA. |
a8f77f7
to
d142d8a
Compare
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
Thanks, CI should be fixed now. I have already sent the signed CLA to cla@trino.io on Jan 14, 2024. |
@cla-bot check |
The cla-bot has been summoned, and re-checked this pull request! |
d142d8a
to
6761b8f
Compare
Description
PR #16674 added support for dropping corrupted Iceberg tables but the Nessie catalog implementation was missing.
Similar as done in the Glue catalog implementation, we load the metadata location from Nessie and extract the table location from it. This allows us to drop the directory where the Iceberg table is located in.
Additional context and related issues
This is a follow up of #16674
Release notes
This is not user-visible or is docs only, and no release notes are required.