You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/12 13:30:42 UTC

[GitHub] [pulsar] eolivelli opened a new pull request, #17592: Issue 17588: Allow deletion of a namespace that was left in deleted status

eolivelli opened a new pull request, #17592:
URL: https://github.com/apache/pulsar/pull/17592

   Fixes #17588
   
   ### Motivation
   
   In Pulsar 2.10.1 with #14730 we introduced a check about concurrent deletion of Namespaces.
   This broke the deletion of namespaces in case of a partial failure.
   
   When you delete a namespace we collect all the resources, then we mark the namespace as "deleted", then we do the deletion.
   if some failure happens and the procedure does not complete, then the namespaces stays in "deleted" status.
   And if you try to delete it again you see the error in #17588 "The namespace is deleted", because getListOfNonPersistentTopics() performs a check about the status of the namespace.  
   
   ### Modifications
   
   Do not check the flag "Policies.deleted" while listing non-persistent topics.è
   
   
   ### Verifying this change
   
   I will add tests
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc-required` 
   (Your PR needs to update docs and you will update later)
   
   - [x] `doc-not-needed` 
   It is a bug fix for a regression
   
   - [ ] `doc` 
   (Your PR contains doc changes)
   
   - [ ] `doc-complete`
   (Docs have been already added)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] eolivelli commented on pull request #17592: Issue 17588: Allow deletion of a namespace that was left in deleted status

Posted by GitBox <gi...@apache.org>.

eolivelli commented on PR #17592:
URL: https://github.com/apache/pulsar/pull/17592#issuecomment-1245229340

   /pulsarbot rerun-failure-checks


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] eolivelli merged pull request #17592: Issue 17588: Allow deletion of a namespace that was left in deleted status

Posted by GitBox <gi...@apache.org>.

eolivelli merged PR #17592:
URL: https://github.com/apache/pulsar/pull/17592


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org