erkanonl opened a new pull request, #14872:
URL: https://github.com/apache/iceberg/pull/14872

   ### What does this change do?
   
   This PR adds lightweight tracking for failed commit attempts in 
BaseCommitService.
   
   Specifically, it:
   
   - Counts the number of failed commit attempts during rewrite actions
   - Captures the exceptions thrown by failed commits for diagnostic purposes
   - Exposes this information via accessor methods for use by callers and tests
   
   ###  Why is this change needed?
   
   BaseCommitService explicitly supports partial progress, but today failed 
commits are only logged.
   This makes it difficult to:
   
   - Programmatically detect partial failures
   - Surface useful diagnostics to callers
   - Write tests that assert failure behavior without relying on log inspection
   
   This change improves observability without altering commit behavior or 
public APIs.
   
   ###  Scope and compatibility
   
   The change is limited to BaseCommitService, which is package-private and 
internal
   No behavioral changes to commit execution or error handling
   No impact on existing public APIs


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to