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 2021/12/20 15:02:11 UTC

[GitHub] [iceberg] zhongyujiang edited a comment on pull request #3772: [Api][Spark]Add cleanExpiredFiles to Actions/ExpireSnapshots

zhongyujiang edited a comment on pull request #3772:
URL: https://github.com/apache/iceberg/pull/3772#issuecomment-997996662


   > That said the ExpireSnapshots method is far more efficient than RemoveOrphans and we should always be encouraging users towards using Expire to delete rather than Remove.
   
   This make sense to me, how about just trigger calculation for **Dataset<Row> originalFiles** before commit and cache it to reduce the probability of such failure.


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