joyhaldar commented on PR #15038: URL: https://github.com/apache/iceberg/pull/15038#issuecomment-3739507886
> How many snapshots you have typically? > > I feel that the code complexity is increasing quite a bit, and I expect that we should not have high enough number of snapshots to make this worth That's a fair point. Most tables have a few thousand snapshots where this won't make a noticeable difference. I was exploring the snapshot expiration code to understand the flow and noticed the duplicate traversal. The change made sense to me at the time, but I understand if the added complexity isn't worth it. Happy to close this if you feel the same way. -- 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]
