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

feat: add ramp up feature to load test client #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alastairtree
Copy link

Hi Tore

Thanks for a great library. Are you interested in adopting this small addition to the console client to allow you to ramp up the number of threads over time? I find it really useful for identifying the optimum number of threads before increasing them fails to increase the number of req/sec the API under test handle.

This change:

  • add a ramp up option to console app "-r" which will repeat the test a number of time
  • it will keep track of the best perf based on number of requests per seconds achieved
  • it increase the number of threads by one if requests per sec has increased
  • it will not increase threads if the last iteration caused an increase in the number of errors
  • reports the optimum number of threads completion to the user
  • it will abandon ramping up after 3 attempts if doing so has lead to no increase in req/sec or to more errors

Thanks

Alastair

- add a ramp up option to console app
- increase the number of threads if requests per sec has increased
- don't increase threads if increase in errors
- keep track of the best perf based on number of threads and report result on completion  to the user
- abandon thread increase after 3 attempts if no increase in req/sec
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