Skip to content

Comments

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

Merged
Xuanwo merged 1 commit intomainfrom
retry-on-rate-limited
Jan 15, 2025
Merged

feat(gcs): Convert TOO_MANY_REQUESTS to retryable Ratelimited#5551
Xuanwo merged 1 commit intomainfrom
retry-on-rate-limited

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?

@Xuanwo Xuanwo requested a review from sundy-li January 15, 2025 09:17
@github-actions github-actions bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jan 15, 2025
@codspeed-hq
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants