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 2022/04/27 18:24:49 UTC

[GitHub] [iceberg] szehon-ho commented on pull request #4651: Core: Optimize to delete the oldest metadata file after commit

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

   I think the first change makes sense, the second one of raising it to the TableOperations base class may not make sense, I am not sure if all TableOperations in the future (particularly REST) will use this mechanism of metadata.json files?  


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