Skip to content
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

[Branch-2.9] Fix delete namespace issue. #14657

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

Technoboy-
Copy link
Contributor

Cherry-pick #14215

Motivation

When we delete namespace with force=false, sometimes there may occur Cannot delete non empty bundle.
Because delete system topic and delete namespace bundles are put into futures, can't keep their order.
We should delete system topic first and then delete namespace bundles.

Modifications

  • Use admin cli to delete system topic.
  • Delete system topic first and then delete namespace bundles
  • Make internalClearZkSources async and the code more readable.

Documentation

  • no-need-doc

@codelipenghui codelipenghui merged commit c4e46a9 into apache:branch-2.9 Mar 11, 2022
@codelipenghui codelipenghui added the cherry-picked/branch-2.9 Archived: 2.9 is end of life label Mar 11, 2022
@Technoboy- Technoboy- deleted the cherry-pick-14215-2-9 branch August 10, 2022 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants