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

   Master Issue: https://github.com/apache/pulsar/issues/16691
   
   ### Motivation
   
   When the channel handles the split event, we should write the split bundle 
to zookeeper instead of `ServiceUnitStateChannel`, since we are using the 
`ServiceUnitStateChannel` to store the bundle ownership.
   
   ### Modifications
   
   * Write the child ownership to `ServiceUnitStateChannel` instead of ZK when 
handling bundle split.
   * Add unit test to cover this change.
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local 
preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR 
description, or else your PR might not get merged. -->
   - [ ] `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: https://github.com/Demogorgon314/pulsar/pull/8
   


-- 
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