Skip to content

[Alternative] Support drop corrupted iceberg table#15435

Closed
krvikash wants to merge 4 commits intotrinodb:masterfrom
krvikash:support-drop-corrupted-iceberg-table-alernative
Closed

[Alternative] Support drop corrupted iceberg table#15435
krvikash wants to merge 4 commits intotrinodb:masterfrom
krvikash:support-drop-corrupted-iceberg-table-alernative

Conversation

@krvikash
Copy link
Contributor

@krvikash krvikash commented Dec 16, 2022

Description

Alternative to #15065. This PR makes use of exception thrown by metadata.getRedirectionAwareTableHandle and Ignore Exception when metadata is not found and force table drop.

NOT FOR REVIEW

Additional context and related issues

NA

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@findepi findepi force-pushed the support-drop-corrupted-iceberg-table-alernative branch from 8a42979 to acea122 Compare December 21, 2022 13:27
@findepi
Copy link
Member

findepi commented Dec 21, 2022

(rebased to resolve conflicts after a force push to master (#15365 (comment)), no other changes)

@findepi
Copy link
Member

findepi commented Dec 21, 2022

Description

Additional context and related issues

Release notes

( ) This is not user-visible or docs only and no release notes are required. ( ) Release notes are required, please propose a release note for me. ( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

If you're not filling the PR template, please remove it from the PR description so that it's obvious to the reader there there isn't any information to look for.

@findepi
Copy link
Member

findepi commented Dec 21, 2022

Instead of empty template, can you replace the PR description with something like "Alternative to #15065. This PR differs in this-particular-way"

@krvikash
Copy link
Contributor Author

Closing. As this PR was just to verify alternative implementation.

@krvikash krvikash closed this Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants