Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

load: Add "weight" load variant #695

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

samvrlewis
Copy link
Contributor

@samvrlewis samvrlewis commented Sep 30, 2022

Adds a weight load variant, which weights an inner load. This is useful in circumstances where it is desireable to artificially inflate or deflate load. One such example is canary deployments, where it might be preferable for a canary to accept less load than its non-canary counterparts.

This change is adapted from the weight implementation that used to exist within tower but was removed (see a496fbf) and an associated unmerged PR (#282).


Some additional discussion around this in #696

@hawkw hawkw requested review from hawkw and olix0r September 30, 2022 18:55
Adds a `weight` load variant, which weights an inner load. This is
useful in circumstances where it is desireable to artificially inflate
or deflate load. One such example is canary deployments, where it might
be preferable for a canary to accept less load than its non-canary
counterparts.

This change is adapted from the weight implementation that used to
exist within tower but was removed (see
tower-rs/tower@a496fbf) and an
associated unmerged PR (tower-rs#282).
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.

1 participant