lxy-9602 opened a new pull request, #397: URL: https://github.com/apache/paimon-cpp/pull/397
<!-- PR titles must follow Conventional Commits: <type>(<optional-scope>): <description> --> ### Purpose <!-- Linking this pull request to the issue --> Linked issue: close #396 Handle `AtomicStore` failures in `RenamingSnapshotCommit` by inspecting the target snapshot: - Return the original error when the target is missing. - Return `false` when the target contains a different snapshot, allowing the existing commit retry loop to rebase. - Treat the commit as successful when the target contains the same snapshot. <!-- What is the purpose of the change --> ### Tests - Added tests for matching, conflicting, and missing target snapshots after an atomic-store error. - Updated the existing lost-response test to expect a successful commit. <!-- List UT and IT cases to verify this change --> ### API and Format <!-- Does this change affect API in include dir or storage format or protocol --> ### Documentation <!-- Does this change introduce a new feature --> ### Generative AI tooling Generated-by: OpenAI Codex (GPT-5) <!-- If generative AI tooling has been used in the process of authoring this patch, please include the phrase: 'Generated-by: ' followed by the name of the tool and its version. If no, write 'No'. Please refer to the [ASF Generative Tooling Guidance](https://www.apache.org/legal/generative-tooling.html) for details. --> -- 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]
