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

   ### Motivation
   
   **background**
   - Once a broker's(we call it `broker-1`) ZK session has expired, the topics 
it owns will be assigned to other brokers
   - And the `broker-1` will get a `BadVersionException`, then the topics will 
be marked `fenced`.
   - Other brokers will load up the topics
   - At this moment there is more than one broker who owns the same topic.
     -  The clients who connected to `broker-1` will get a fenced error.
   
   Issue: the topics on `broker-1` will not be unloaded until it restarts, so 
we'd better change the default behavior of the action when ZK session expires 
to `shutdown`, which will solve the issue automatically. We can set it back to 
`reconnect` after we improve the behavior to unload the topic automatically in 
the future.
   
   ### Modifications
   
   Change default behavior when ZK session expired to `shutdown`.
   
   ### 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: commits-unsubscr...@pulsar.apache.org

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

Reply via email to