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

Accept some differing units for throttling #1103

Merged
merged 2 commits into from
Nov 3, 2020
Merged

Accept some differing units for throttling #1103

merged 2 commits into from
Nov 3, 2020

Commits on Nov 3, 2020

  1. Accept some differing units for throttling 

    With elastic#1100 we introduced more flexible throttling which requires that
the unit in which requests are throttled and the unit in which they are
reported, are aligned. This causes issues at the moment with scroll
requests which are throttled in ops/s but reported in pages/s.

With this commit we correct the unit mismatch for this special case in
order to stay backwards-compatible with older versions of Rally. We also
ensure that throttling is preserved in test mode so our integration
tests can spot such issues in the future.

Relates elastic#1100
    @danielmitterdorfer
    danielmitterdorfer committed Nov 3, 2020
    Configuration menu
    Copy the full SHA
    fb0cce1 View commit details
    Browse the repository at this point in the history

  2. Change weight for any unit as long as the target is ops/s

    @danielmitterdorfer
    danielmitterdorfer committed Nov 3, 2020
    Configuration menu
    Copy the full SHA
    309f6d6 View commit details
    Browse the repository at this point in the history