Re: [PR] Core: Preserve concurrent snapshots on replace transaction retry [iceberg]

["wombatu-kun (via GitHub)" <gi...@apache.org>]

wombatu-kun commented on PR #16943:
URL: https://github.com/apache/iceberg/pull/16943#issuecomment-5086617118

   @nastra @amogh-jahagirdar this one has been sitting since June 24 with no review at all. CI is green (56/56) and it still merges cleanly on current `main`, so it is only waiting on a reviewer.
   
   It fixes silent snapshot loss: when two `replace` / `createOrReplace` transactions race, the loser re-commits replacement metadata built from its stale base, so the concurrent writer's snapshot disappears from history (#16942). This hits the catalogs that rewrite full metadata (Hadoop, Hive, Glue, JDBC, Nessie); REST merges server-side and is unaffected.
   
   Tagging you two because you handled the closest prior work in this path (#11671). A look whenever you have time would be much appreciated.
   


-- 
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@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org