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

Fixed double delete on a namespace #6713

Merged
merged 2 commits into from
Apr 14, 2020
Merged

Conversation

merlimat
Copy link
Contributor

Motivation

Creating and deleting the same namespace twice is raising an error in the cleanup phase. We should ignore the no-node exception during cleanup.

@merlimat merlimat added type/bug The PR fixed a bug or issue reported a bug release/2.5.2 labels Apr 11, 2020
@merlimat merlimat added this to the 2.6.0 milestone Apr 11, 2020
@merlimat merlimat self-assigned this Apr 11, 2020
@sijie
Copy link
Member

sijie commented Apr 12, 2020

@merlimat can you rebase the pull request?

@sijie
Copy link
Member

sijie commented Apr 14, 2020

/pulsarbot run-failure-checks

@merlimat merlimat merged commit 88f401c into apache:master Apr 14, 2020
cdbartholomew pushed a commit to kafkaesque-io/pulsar that referenced this pull request Apr 18, 2020
cdbartholomew pushed a commit to kafkaesque-io/pulsar that referenced this pull request Apr 29, 2020
addisonj pushed a commit to instructure/pulsar that referenced this pull request May 7, 2020
jiazhai pushed a commit that referenced this pull request May 8, 2020
jerrypeng pushed a commit to jerrypeng/incubator-pulsar that referenced this pull request May 15, 2020
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/2.5.2 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants