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

Allow RPS > 1000 #742

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Allow RPS > 1000 #742

merged 1 commit into from
Feb 9, 2024

Conversation

dhschall
Copy link
Contributor

If the RPS is higher that 1000 the Invoker will have a delay of 0 which causes to not issue any requests. This patch allows RPS up to 1M and issues a error if the targetRPS it to high

@dhschall dhschall self-assigned this Oct 25, 2023
@ustiugov
Copy link
Member

please sign the commit and fix the broken links in the READMEs

If the RPS is higher that 1000 the Invoker will have a delay
of 0 which causes to not issue any request. This patch allows
RPS up to 1M and issues a error if the target RPS it to high.

Signed-off-by: David Schall <[email protected]>
@shyamjesal shyamjesal merged commit b6058b8 into main Feb 9, 2024
148 of 150 checks passed
@shyamjesal shyamjesal deleted the dhschall-patch-1 branch February 9, 2024 15:23
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.

3 participants