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

Measure time until last response chunk #1823

Conversation

danielmitterdorfer
Copy link
Member

Rally uses timers internally to measure when a request started and when it has completed. However, it has stopped measuring as soon as the first byte of the response has been received instead of waiting for the full response. This can skew service time measurement if the response is large and Elasticsearch streams it.

With this commit we wait until the last chunk of the response has been received. This results in a more realistic service time metric in these cases.

Closes #1822

Rally uses timers internally to measure when a request started and when
it has completed. However, it has stopped measuring as soon as the first
byte of the response has been received instead of waiting for the full
response. This can skew service time measurement if the response is
large and Elasticsearch streams it.

With this commit we wait until the last chunk of the response has been
received. This results in a more realistic service time metric in these
cases.

Closes elastic#1822
@danielmitterdorfer danielmitterdorfer added bug Something's wrong :Load Driver Changes that affect the core of the load driver such as scheduling, the measurement approach etc. labels Jan 15, 2024
@danielmitterdorfer danielmitterdorfer added this to the 2.x milestone Jan 15, 2024
@danielmitterdorfer danielmitterdorfer self-assigned this Jan 15, 2024
Copy link

@dpifke-elastic dpifke-elastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@danielmitterdorfer danielmitterdorfer merged commit 3833d53 into elastic:master Jan 17, 2024
15 checks passed
@danielmitterdorfer danielmitterdorfer deleted the measure-time-to-last-chunk branch January 17, 2024 06:43
@gareth-ellis gareth-ellis modified the milestones: 2.x, 2.11 May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something's wrong :Load Driver Changes that affect the core of the load driver such as scheduling, the measurement approach etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Service time measurement stops at first response byte
3 participants