Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

perf(round_robin) ring balancer with no hashing feats #123

Merged
merged 5 commits into from Apr 2, 2021

Conversation

locao
Copy link
Contributor

@locao locao commented Mar 9, 2021

This commit adds a round-robin algorithm with the same features as the previous ring algorithm, but with no consistent-hashing support.

@locao locao force-pushed the perf/ring_balancer_cleanup branch 4 times, most recently from edd40e2 to 9abccbe Compare March 10, 2021 14:34
spec/balancer/round_robin_spec.lua Outdated Show resolved Hide resolved
src/resty/dns/balancer/round_robin.lua Outdated Show resolved Hide resolved
src/resty/dns/balancer/round_robin.lua Outdated Show resolved Hide resolved
@locao locao force-pushed the perf/ring_balancer_cleanup branch 2 times, most recently from 9b615f8 to 2a821a2 Compare March 31, 2021 16:06
Copy link
Member

@kikito kikito left a comment

Choose a reason for hiding this comment

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

LGTM, I think @javierguerragiraldez should give this a final approval

@locao locao force-pushed the perf/ring_balancer_cleanup branch from 2a821a2 to 1d2f1ac Compare April 1, 2021 14:09
This commit adds a round-robin algorithm with the same features as the
previous ring algorithm, but with no consistent-hashing support.
@locao locao force-pushed the perf/ring_balancer_cleanup branch 3 times, most recently from f0c5dc1 to 80aa2c7 Compare April 1, 2021 19:34
@locao locao force-pushed the perf/ring_balancer_cleanup branch from 80aa2c7 to ca98b82 Compare April 2, 2021 01:02
@locao locao merged commit 4ea417e into master Apr 2, 2021
@locao locao deleted the perf/ring_balancer_cleanup branch April 2, 2021 01:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants