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

Automated cherry pick of #118601: priority & fairness: support dynamic max seats #119573

Commits on Jul 25, 2023

  1. priority & fairness: support dynamically configuring work estimator m…

    …ax seats
    
    Max seats from prioriy & fairness work estimator is now min(0.15 x
    nominalCL, nominalCL/handSize)
    
    'Max seats' calculated by work estimator is currently hard coded to 10.
    When using lower values for --max-requests-inflight, a single
    LIST request taking up 10 seats could end up using all if not most seats in
    the priority level. This change updates the default work estimator
    config such that 'max seats' is at most 10% of the
    maximum concurrency limit for a priority level, with an upper limit of 10.
    This ensures seats taken from LIST request is proportional to the total
    available seats.
    
    Signed-off-by: Andrew Sy Kim <andrewsy@google.com>
    andrewsykim committed Jul 25, 2023
    Configuration menu
    Copy the full SHA
    6c91cf9 View commit details
    Browse the repository at this point in the history