store: support hedged requests #7860

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

Merged

GiedriusS merged 12 commits into thanos-io:main from milinddethe15:support-hedged-requests

Nov 14, 2024

Contributor

milinddethe15 commented Oct 26, 2024 •

edited

Loading

fixes #6712

I added CHANGELOG entry for this change.
Change is not relevant to the end user.

Changes

Verification


          support hedged requests in store

3520b2f

Signed-off-by: milinddethe15 <[email protected]>

pull-request-size bot added the size/M label


          Merge branch 'main' into support-hedged-requests

f6c255e

GiedriusS reviewed

View reviewed changes

pkg/exthttp/transport.go Outdated Show resolved Hide resolved


          hedged roundtripper with tdigest for dynamic delay

e6b2a73

Signed-off-by: milinddethe15 <[email protected]>

pull-request-size bot added size/L and removed size/M labels

milinddethe15 and others added 2 commits

November 8, 2024 01:32


          Merge branch 'main' into support-hedged-requests

825d456


          refactor struct and fix lint

59c5a80

Signed-off-by: milinddethe15 <[email protected]>

saswatamcode reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

saswatamcode reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved


          Improve hedging implementation

2ac08a8

Signed-off-by: milinddethe15 <[email protected]>

GiedriusS reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

GiedriusS reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

GiedriusS reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

GiedriusS reviewed

View reviewed changes

pkg/exthttp/hedging.go Show resolved Hide resolved

GiedriusS reviewed

View reviewed changes

pkg/exthttp/hedging.go Outdated Show resolved Hide resolved

milinddethe15 and others added 5 commits

November 12, 2024 06:37


          Improved hedging implementation

2a3b2cc

Signed-off-by: milinddethe15 <[email protected]>


          Merge branch 'main' into support-hedged-requests

f29b99b


          Update store doc

c3895b2

Signed-off-by: milinddethe15 <[email protected]>


          Merge branch 'main' into support-hedged-requests

2790ab9


          fix white space

29fb0e4

Signed-off-by: milinddethe15 <[email protected]>

GiedriusS reviewed

View reviewed changes

cmd/thanos/store.go Show resolved Hide resolved


          add enabled field

753db7f

Signed-off-by: milinddethe15 <[email protected]>

GiedriusS approved these changes

View reviewed changes

GiedriusS marked this pull request as ready for review

November 14, 2024 13:46

saswatamcode approved these changes

View reviewed changes

Member

saswatamcode left a comment

🚀

GiedriusS merged commit dc4c49f into thanos-io:main

22 checks passed

Contributor

yeya24 commented Nov 17, 2024

It would be nice to add a changelog! Great work

yeya24 mentioned this pull request

Store Gateway: Support hedged requests to object store cortexproject/cortex#6343

Closed

yeya24 mentioned this pull request

Cut first release candidate for v0.37.0 #7921

Merged

2 tasks

BrewTestBot mentioned this pull request

thanos 0.37.0 Homebrew/homebrew-core#198927

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels