[PR] [python] Fix update_by_row_id commit cleanup semantics [paimon]

["QuakeWang (via GitHub)" <gi...@apache.org>]

QuakeWang opened a new pull request, #8522:
URL: https://github.com/apache/paimon/pull/8522

   ### Purpose
   
   Fix `update_by_row_id` commit lifecycle handling so failures around commit cleanup follow the same transaction boundary as `merge_into`:
   
   - abort pending commit messages only when failure happens before commit starts
   - do not abort once commit has started
   - log commit/abort close failures as warnings so they do not override a successful commit or mask the original commit failure
   
   This keeps the change scoped to `update_by_row_id` commit/abort/close semantics and adds regression coverage for the failure boundaries.
   
   ### Tests
   
   - `python -m pytest pypaimon/tests/ray_update_by_row_id_test.py -q`
   - `git diff --check`
   


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [python] Fix update_by_row_id commit cleanup semantics [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi merged PR #8522:
URL: https://github.com/apache/paimon/pull/8522


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [python] Fix update_by_row_id commit cleanup semantics [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi commented on PR #8522:
URL: https://github.com/apache/paimon/pull/8522#issuecomment-4930844762

   +1


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org