Remove unnecessary Failure check in doCommit path #8120
Conversation
|
@jackye1995 would you have some cycles to look into this, since you're the original author? |
|
@pvary Do you have time to dig into this one? |
|
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. |
2 participants
With the introduction of retries, do we ever get into a Failure state in
commitStatusin catalog? I think not. This PR attempts to clean up code that's never needed.