arnaudbriche opened a new pull request, #401: URL: https://github.com/apache/iceberg-go/pull/401
Hi @zeroshade, As discussed on a previous PR, I'm trying to implement snapshots expiration now. This is not complete, but I wanted to discuss it with you before I further work on it. If the design looks ok to you, here are a few technical questions regarding snapshot expiration: - currently, the snapshots field of the produced metadata file looks ok (expired snapshots are not there), but the snapshot-log field still contains entries for every operation since table creation. Is is expected ? I don't find an appropriate answer in the spec. - should we handle expired data and metadata files deletion here ? - the produced metadata file does not contains any new snapshot (not snapshot is created for the expire snapshots operation). I cannot find out what the spec says about it. Most probably a new snapshot must be created. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
