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

   ### Motivation
   
   Avoid the bestBroker choice broker exceed candidates when selectBroker in 
LeastResourceUsageWithWeight.
   When loadManager leader running a period of time,   
`brokerAvgResourceUsageWithWeight` will contain almost all brokers, the judge 
will cause we selected broker exceed candidates, it's out of expectation.
   
   ### Modifications
   
   1. check if `candidates` contain the broker before add the broker to 
`bestBrokers`.
   
   
   ### Documentation
   
   - [X] `doc-not-needed` 
   (Please explain why)
     


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