badalprasadsingh commented on PR #2194: URL: https://github.com/apache/iceberg-go/pull/2194#issuecomment-6288886783
Confirmed, thanks. I reworked it rather than patching the set: the commit-wide exemption is gone now. Each delete, when staged, now replays the staged snapshots it builds on, oldest first: their added data files minus their tombstoned files. Inherited manifests are skipped, so head files are still re-checked against the refreshed head each attempt. Your sequence fails when the third row delta is staged, with the missing data files error, so only the add and rewrite commit. This also rejects a dv staged before its file's add (lower sequence number) and a dv for a head file removed earlier. -- 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]
