dragonknightk commented on issue #2604: URL: https://github.com/apache/iceberg-python/issues/2604#issuecomment-6519916202
Hi everyone, thank you all for the thoughtful discussion here! (´｡• ᵕ •｡`) I've opened #4201 based on the approach you outlined: the expiration is committed first, and only then are the files referenced solely by the expired snapshots deleted. Since deletion happens after the commit, the table isn't held up while files are being removed. It's opt-in via `clean_expired_files()`, and CI is all green. ʕ•ᴥ•ʔ @Fokko @Anton-Tarazi @ForeverAngry since this direction came from you, I'd be really grateful if you could take a look whenever you have a moment. No rush at all, and any feedback is very welcome! (๑•̀ㅂ•́)و✧ -- 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]
