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

Record object storage request latencies #5521

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Conversation

guilload
Copy link
Member

Description

Per title.

How was this PR tested?

Ran quickwit-storage test suite

@guilload guilload force-pushed the guilload/storage-latency branch from 8ab8131 to 38a8e89 Compare October 23, 2024 21:20
Copy link

github-actions bot commented Oct 23, 2024

On SSD:

Average search latency is 1.01x that of the reference (lower is better).
Ref run id: 4045, ref commit: 105aa7d
Link

On GCS:

Average search latency is 0.982x that of the reference (lower is better).
Ref run id: 4087, ref commit: 826f10f
Link

@guilload guilload force-pushed the guilload/storage-latency branch from 38a8e89 to d628047 Compare October 23, 2024 22:18
@guilload guilload marked this pull request as ready for review October 23, 2024 22:20
@guilload guilload force-pushed the guilload/storage-latency branch from d628047 to df9822d Compare October 30, 2024 14:02
@guilload guilload force-pushed the guilload/storage-latency branch from df9822d to c23c150 Compare October 30, 2024 14:25
@guilload guilload merged commit 58944fc into main Oct 30, 2024
5 checks passed
@guilload guilload deleted the guilload/storage-latency branch October 30, 2024 15:20
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