Skip to content

[HUDI-5563] Check table exist before drop table #7679

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.

Sign up for GitHub

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

Jump to bottom
Merged
YannByron merged 2 commits into from
Jan 31, 2023
Merged

[HUDI-5563] Check table exist before drop table #7679

YannByron merged 2 commits into from
Jan 31, 2023

Conversation

@Zouxxyy
Copy link
Contributor

@Zouxxyy Zouxxyy commented Jan 16, 2023 •
edited
Loading

Change Logs

Check table exist when drop table to avoid exec drop table if exists non_existent_table failure which caused by Exception thrown by sparkAdapter.isHoodieTable

Impact

Drop table if exists will not fail when table does not exist

Risk level (write none, low medium or high below)

none

Documentation Update

None

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@Zouxxyy Zouxxyy changed the title [HUDI-5563] check table exist before drop table [HUDI-5563] Check table exist before drop table Jan 16, 2023
@danny0405 danny0405 added priority:medium Moderate impact; usability gaps area:sql SQL interfaces labels Jan 17, 2023
@YannByron
Copy link
Contributor

YannByron commented Jan 28, 2023

@hudi-bot run azure

1 similar comment
@Zouxxyy
Copy link
Contributor Author

Zouxxyy commented Jan 28, 2023

@hudi-bot run azure

@hudi-bot
Copy link
Collaborator

hudi-bot commented Jan 30, 2023

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@YannByron YannByron merged commit 1a72f50 into apache:master Jan 31, 2023
@Zouxxyy Zouxxyy deleted the fix-drop-table branch January 31, 2023 01:47
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Jan 31, 2023
@Zouxxyy
[HUDI-5563] Check table exist before drop table (apache#7679)
3fb07e0
yihua pushed a commit that referenced this pull request Jan 31, 2023
@Zouxxyy @yihua
[HUDI-5563] Check table exist before drop table (#7679)
4768408
@yihua yihua added the priority:blocker Production down; release blocker label Feb 5, 2023
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Mar 22, 2023
@Zouxxyy @nsivabalan
[HUDI-5563] Check table exist before drop table (apache#7679)
e75f830
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Apr 5, 2023
@Zouxxyy
[HUDI-5563] Check table exist before drop table (apache#7679)
56c3860
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@YannByron YannByron YannByron approved these changes

@XuQianJin-Stars XuQianJin-Stars XuQianJin-Stars approved these changes

Assignees

@XuQianJin-Stars XuQianJin-Stars

Labels

area:sql SQL interfaces priority:blocker Production down; release blocker priority:medium Moderate impact; usability gaps

Projects

Hudi PR Support
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Zouxxyy @YannByron @hudi-bot @XuQianJin-Stars @yihua @danny0405