lordcheng10 commented on PR #16552:
URL: https://github.com/apache/pulsar/pull/16552#issuecomment-1185112645

   > > > @lordcheng10 Could you please provide more details about the BUG? so 
that the reviewers can easily understand what the issue is that the PR wants to 
fix.
   > > 
   > > 
   > > Sorry, I didn't describe clearly. @codelipenghui
   > > I'll give an example to illustrate:
   > > 
   > > 1. Configure loadBalancerNamespaceMaximumBundles=128
   > > 2. The current number of bundles in the namespace is 120
   > > 3. According to the current logic, after executing the method 
findBundlesToSplit, 40 bundles may be found that need to be split;
   > > 4. After these bundles, the number of bundles in the namespace will 
become 120+40=160 > loadBalancerNamespaceMaximumBundles=128
   > 
   > Make sense.
   
   When judging whether the number of bundles exceeds the configuration, it is 
necessary to add the already selected bundles. @codelipenghui @Technoboy- 


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