Skip to content

fix: retry on 409 with AWS S3#6742

Merged
Xuanwo merged 2 commits intoapache:mainfrom
wolfv:retry-409
Oct 28, 2025
Merged

fix: retry on 409 with AWS S3#6742
Xuanwo merged 2 commits intoapache:mainfrom
wolfv:retry-409

Conversation

@wolfv
Copy link
Copy Markdown
Contributor

@wolfv wolfv commented Oct 27, 2025

Trying to fix #6741

I am not sure if we should do the same for other backends?

Also not sure if the error code is correct?

@wolfv wolfv requested a review from Xuanwo as a code owner October 27, 2025 17:31
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Oct 27, 2025
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Oct 27, 2025

I am not sure if we should do the same for other backends?

We might need to go through the docs to check each one, but it’s fine with me to merge this one first.

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you for working on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 28, 2025
@Xuanwo Xuanwo merged commit 1a7dd56 into apache:main Oct 28, 2025
86 checks passed
@wolfv wolfv deleted the retry-409 branch October 28, 2025 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: retry on 409 when condition not matched

2 participants