[GitHub] [iceberg] RussellSpitzer commented on issue #2221: Spark: Extend expire_snapshots procedure with an optional arg for snapshot ids

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

RussellSpitzer commented on issue #2221:
URL: https://github.com/apache/iceberg/issues/2221#issuecomment-774365830


   I think that's reasonable, better than full subquery support :) One thing I worry about is we have a bunch of code which relies on the snapshot chain being fully connected. So if we have Snapshots A, B, C and remove B, will we have issues?
   
   Also this means we will have to extend the parser to also handle Array types I think


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



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