Skip to content

fix: fix sql catalog drop table#853

Merged
Xuanwo merged 1 commit into
apache:mainfrom
Li0k:li0k/iceberg_fix_drop_table
Dec 27, 2024
Merged

fix: fix sql catalog drop table#853
Xuanwo merged 1 commit into
apache:mainfrom
Li0k:li0k/iceberg_fix_drop_table

Conversation

@Li0k

@Li0k Li0k commented Dec 27, 2024

Copy link
Copy Markdown
Contributor

This PR fixes the wrong filter condition in drop table

  • Fix sql
  • Add UT to validate the correctness of the drop table

@Li0k Li0k changed the title fix(catalog): fix sql catalog drop table fix: fix sql catalog drop table Dec 27, 2024
@Li0k

Li0k commented Dec 27, 2024

Copy link
Copy Markdown
Contributor Author

cc @Xuanwo @liurenjie1024 @ZENOTME

@Xuanwo Xuanwo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Li0k for the fix!

@Xuanwo Xuanwo merged commit e5fffe4 into apache:main Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants