Skip to content

Conversation

@chenBright
Copy link
Contributor

What problem does this PR solve?

Issue Number:

Problem Summary:

  1. WeightedRandomizedLoadBalancer returns 0 without server.
  2. The condition for finding an available server among the remaining servers is wrong.

What is changed and the side effects?

Changed:

Side effects:

  • Performance effects:

  • Breaking backward compatibility:


Check List:

@wwbmmm
Copy link
Contributor

wwbmmm commented Sep 28, 2025

LGTM

@chenBright chenBright merged commit 2b757e9 into apache:master Sep 28, 2025
16 checks passed
@chenBright chenBright deleted the fix_wr branch September 28, 2025 12:42
chenBright added a commit to chenBright/brpc that referenced this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants