[GitHub] [iceberg] szehon-ho commented on pull request #4141: Add unit test for ExpireSnapshot with DeleteFile

[GitBox <gi...@apache.org>]

szehon-ho commented on PR #4141:
URL: https://github.com/apache/iceberg/pull/4141#issuecomment-1287267038

   Hi, when i tested this, i think the issue is not that expireSnapshot doesnt remove delete files.  Its that the delete files dont get removed from current snapshot.   Issue is described here:  https://github.com/apache/iceberg/issues/4127
   
   
   Can you check that?  You can query 'files' table, does that have the delete files?  If so then its still on current snapshot).
   
   
   Im working on a design for fix for this, but havent had time yet.  Hopefully next week will put a design doc up.


-- 
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