Skip to content

feat: load balance by least connections#3304

Merged
spacewander merged 1 commit into
apache:masterfrom
spacewander:least_conn
Jan 18, 2021
Merged

feat: load balance by least connections#3304
spacewander merged 1 commit into
apache:masterfrom
spacewander:least_conn

Conversation

@spacewander

Copy link
Copy Markdown
Member

Fix #3086

Signed-off-by: spacewander [email protected]

What this PR does / why we need it:

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@spacewander
spacewander marked this pull request as ready for review January 15, 2021 09:58
@sysulq

sysulq commented Jan 15, 2021

Copy link
Copy Markdown
Contributor

What about p2c with least loaded, maybe we could benchmark this two implementations?

@spacewander

Copy link
Copy Markdown
Member Author

Yes, we can add a new option to enable p2c.

@lilien1010 lilien1010 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
awesome feature

@spacewander
spacewander merged commit f349cc0 into apache:master Jan 18, 2021
@spacewander
spacewander deleted the least_conn branch January 18, 2021 02:26
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.

request help: Support load balance by least connections

4 participants