arnaudbriche commented on PR #401:
URL: https://github.com/apache/iceberg-go/pull/401#issuecomment-2847133044

   Hi @zeroshade 
   
   Sure, I'll add tests.
   Regarding these 3 points, can you provide guidance/opinions ?
   
   - 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

Reply via email to