Skip to content

HDDS-14968. Concurrent S3 Conditional PUT Commit Conflict Detection#10043

Open
peterxcli wants to merge 4 commits intoapache:masterfrom
peterxcli:fix/handle-concurrent-conditional-write-commit-conflict
Open

HDDS-14968. Concurrent S3 Conditional PUT Commit Conflict Detection#10043
peterxcli wants to merge 4 commits intoapache:masterfrom
peterxcli:fix/handle-concurrent-conditional-write-commit-conflict

Conversation

@peterxcli
Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

  • Make mormal Atomic Rewrite return ATOMIC_WRITE_CONFLICT when detect conflict when committing key
  • The Change in the OMKeyRequest fix the concurrent conflict detection of put-if-absent request

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14968

How was this patch tested?

@peterxcli peterxcli changed the title Handle concurrent conditional write commit conflict HDDS-14968. Concurrent S3 Conditional PUT Commit Conflict Detection Apr 4, 2026
@peterxcli peterxcli self-assigned this Apr 4, 2026
@peterxcli peterxcli requested a review from ivandika3 April 4, 2026 16:47
@peterxcli peterxcli marked this pull request as ready for review April 4, 2026 16:48
@peterxcli
Copy link
Copy Markdown
Member Author

It's hard to write deterministic e2e test for the commit conflict case, even the atomic rewrite itself doesn't have one.

@peterxcli peterxcli requested a review from sodonnel April 4, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant