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

   ### Motivation
   
   **Background**
   
   Broker will delete topics automaticailly if the local cluster is removed 
from namespace-level policies.
   
   The PR https://github.com/apache/pulsar/pull/23313 made topic deletion 
relies on reading namespace level policies, which leads to the issue that topic 
deletion fails if the local cluster has been removed from namespace-level 
policies.
   
   **before #23313, the steps that deletes topics**
   - disconnect clients
   - unsubscribe all subscriptions
   - delete topic
   
   **after #23313, the steps that deletes topics**
   - disconnect clients
   - unsubscribe all subscriptions
     - **(Highlight)** query namespace policies and topic level policies, which 
will fail since the local cluster has been removed already. See also 
https://github.com/apache/pulsar/pull/23313/files#diff-5edf14cc6f25857d0cfdd26b2d3b3141230ecfb0dfa95aebf7583fd76ede4c4bR1237-R1238
   - delete topic
   
   ### Modifications
   
   Modify the step "unsubscribe all subscriptions", instead of build new 
managed ledger configuration, use the compeletely built 
config(`managedLedger.config`), which avoids to access policies.
   
   
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: x
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to