[GitHub] [iceberg] ajantha-bhat commented on pull request #3718: Spark: Support passing specific list of snapshot id for ExpireSnapshots procedure

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

ajantha-bhat commented on pull request #3718:
URL: https://github.com/apache/iceberg/pull/3718#issuecomment-992079054


   > @ajantha-bhat, what is the use case for exposing this?
   
   > Expiring individual snapshots by ID is not a recommended operation because it can break certain operations that use table history. Unless there's a reason to expose this to SQL users, I would rather not.
   
   @rdblue 
   a) I am not newly exposing this feature to the user, it was already exposed in the Spark Actions. I am just supporting from SQL also to be in sync.
   b) I see the code to handle snapshot history log file also present in expireSnapshots. So, can you elaborate what operations it can break ? If it is really issue, do we need to handle this for spark actions also ?
   


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