[GitHub] [incubator-iceberg] aokolnychyi commented on issue #780: expire table metadata with ExpireSnapshots

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

aokolnychyi commented on issue #780: expire table metadata with ExpireSnapshots
URL: https://github.com/apache/incubator-iceberg/pull/780#issuecomment-583044458
 
 
   I think this PR is a follow-up on #631. 
   
   As far as I remember, we planned that the main place to remove old metadata JSON files will be while expiring snapshots. After a quick look, it seems we won't delete metadata files unless the user explicitly calls a method in `ExpireSnapshots`. I would, probably, always reduce the number of metadata JSON files to a smaller number and expose a method to overwrite the number of files to keep. For example, we can have a table property that defines the default number of files to keep while expiring snapshots and a method to override this value programmatically.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org