Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(gcs): Convert TOO_MANY_REQUESTS to retryable Ratelimited #5551

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jan 15, 2025

Which issue does this PR close?

gcs's returning TOO_MANY_REQUESTS should be treated at retryable Ratelimited instead.

Rationale for this change

None

What changes are included in this PR?

Are there any user-facing changes?

…imited

Signed-off-by: Xuanwo <github@xuanwo.io>
Copy link

codspeed-hq bot commented Jan 15, 2025

CodSpeed Performance Report

Merging #5551 will not alter performance

Comparing retry-on-rate-limited (4cb4de6) with main (6734787)

Summary

✅ 73 untouched benchmarks

@Xuanwo Xuanwo merged commit 78b6a9f into main Jan 15, 2025
95 checks passed
@Xuanwo Xuanwo deleted the retry-on-rate-limited branch January 15, 2025 09:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants