feat: Add configurable commit retry logic for snapshot and transaction operations#2794
feat: Add configurable commit retry logic for snapshot and transaction operations#2794KazuhitoT wants to merge 12 commits intoapache:mainfrom
Conversation
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
|
Hey! ✋ Any updates on this? |
|
@KazuhitoT , congrats on the great work! Do you plan to reopen this PR? I’m looking forward to this feature! |
|
Thanks. |
Rationale for this change
This change adds commit retry handling for table and snapshot operations.
Related issue: #269
Are these changes tested?
Yes. Tests for retry utilities and commit retry behaviours are included.
Are there any user-facing changes?
Yes. Users can handle commit retry behavior via table properties:
commit.retry.num-retriescommit.retry.min-wait-mscommit.retry.max-wait-mscommit.retry.total-timeout-ms