You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2020/02/17 19:36:48 UTC
[GitHub] [incubator-iceberg] yathindranath edited a comment on issue #780:
expire table metadata with ExpireSnapshots
yathindranath edited a comment on issue #780: expire table metadata with ExpireSnapshots
URL: https://github.com/apache/incubator-iceberg/pull/780#issuecomment-587130697
@rdblue The default value of `TableProperties.METADATA_DELETE_AFTER_COMMIT_ENABLED` is false and if we enable this at a later point, can leave orphan metadata files. I believe, it's better to do clean up of old metadata files during snapshot expiration and remove the current logic from the commit what we added in #631.
@aokolnychyi I agree, we can have table properties for snapshot retention interval, number snapshots to retain and number of metadata files to retain. We can create a different PR for expiration with table properties.
Sorry, closed the PR accidentally, reopened it.
----------------------------------------------------------------
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