amoro icon indicating copy to clipboard operation
amoro copied to clipboard

[Bug]: Allow drop non-empty external catalogs

Open link3280 opened this issue 1 year ago • 0 comments

Search before asking

  • [X] I have searched in the issues and found no similar issues.

What would you like to be improved?

Currently, it's forbidden to remove a non-empty catalog if a catalog has managed tables. However, the implementation actually checks all tables, not only the managed ones. As a result, external catalogs can never be removed.

https://github.com/apache/amoro/blob/84e095b6e28a1730d2709daedf0acbffad7b12fc/ams/server/src/main/java/com/netease/arctic/server/table/DefaultTableService.java#L233-L237

How should we improve?

No response

Are you willing to submit PR?

  • [X] Yes I am willing to submit a PR!

Subtasks

No response

Code of Conduct

link3280 avatar Apr 11 '24 08:04 link3280