Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add query retries in case druid returns 429 #3806

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
merged 3 commits into from
Jan 10, 2024
Merged

add query retries in case druid returns 429 #3806

merged 3 commits into from
Jan 10, 2024

Conversation

pjain1
Copy link
Member

@pjain1 pjain1 commented Jan 9, 2024 •
edited
Loading

No description provided.

@pjain1 pjain1 requested a review from begelundmuller January 9, 2024 14:37
begelundmuller
begelundmuller requested changes Jan 9, 2024
View reviewed changes
@pjain1 pjain1 requested a review from begelundmuller January 10, 2024 07:29
@k-anshul k-anshul added the blocker A release blocker issue that should be resolved before a new release label Jan 10, 2024
begelundmuller
begelundmuller approved these changes Jan 10, 2024
View reviewed changes
Copy link
Contributor

@begelundmuller begelundmuller left a comment

Choose a reason for hiding this comment

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

LGTM

@begelundmuller begelundmuller merged commit 3ca032d into main Jan 10, 2024
@begelundmuller begelundmuller deleted the retry_druid branch January 10, 2024 13:41
k-anshul pushed a commit that referenced this pull request Jan 10, 2024
@pjain1 @k-anshul
Add query retries in case druid returns 429 (#3806)
3ee5d89 
* add query retries in case druid returns 429

* use retrier lib

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

@begelundmuller begelundmuller begelundmuller approved these changes

Assignees
No one assigned
Labels
blocker A release blocker issue that should be resolved before a new release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pjain1 @begelundmuller @k-anshul