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.

Sign up for GitHub

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

Jump to bottom

chore: Client change for using request priority to enable flow control #1900

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

chore: Client change for using request priority to enable flow control #1900

wants to merge 4 commits into from

Commits on Sep 11, 2023

  1. Client change for using request priority to enable flow control 

    In this stage, client needs to do 3 things:
1. Always create flow control callable regardless of client flag
2. Make sure the entire callable behave like no-op if RateLimitInfo is not present.
3. Make sure client runs flow control as long as RateLimitInfo is present, regardless of client flag.

Meanwhile, setting client flag would still set the feature flag.

On server side, we'll compute and return RateLimitInfo if server side sees priority is low.
    @kongweihan
    kongweihan committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    761bab7 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'origin/main' into kongwh-request-priori… 

    …ty-patch-1

Change-Id: Ib9f2520b70cc570d76dbe04e5ecd1ae4a28ac8c5
    @igorbernstein2
    igorbernstein2 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    722e5ee View commit details
    Browse the repository at this point in the history

  3. remove log noise 

    Change-Id: I721ad64ce49b0310eb44d86e3b171ab2bef46437
    @igorbernstein2
    igorbernstein2 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    5f4f32f View commit details
    Browse the repository at this point in the history

  4. fix missing call to outer observer 

    Change-Id: Ibda6daed72f0f267ad741836701848f33c2d5377
    @igorbernstein2
    igorbernstein2 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    d128fce View commit details
    Browse the repository at this point in the history