Demogorgon314 opened a new pull request, #20878: URL: https://github.com/apache/pulsar/pull/20878
(cherry picked from commit 3f637680bbbc268d4801172d3212279ace38c4d4) ### Motivation The bundle ownership assignment logic doesn't know the previous unloaded broker when unloading happens. It might assign the bundle to the same broker. In this case, it might cause an infinite bundle unloading loop. To resolve this issue, we can check the destination broker when doing the `doLoadShedding` stage. When the destination broker is the same as the current owner broker, we can skip this unload, and if it is different, we set the new owner in this stage. ### Modifications * Transfer the bundle to the new owner when needed to unload. ### Verifying this change See the `ModularLoadManagerImplTest#testLoadShedding` unit test. ### 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 --> -- 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]
