[GitHub] [iceberg] rdblue commented on pull request #2752: Spark MicroBatch read - Ignore replace snapshots and add Spark option to skip delete snapshots

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

rdblue commented on pull request #2752:
URL: https://github.com/apache/iceberg/pull/2752#issuecomment-877413993


   We try not to use namespaces in the read options to keep them simple. This one is a bit odd because `ignore-deletes` could be misinterpreted easily -- we don't want anyone to think they can read a table without applying delete files, for example. So I want to make it clear that this is streaming and that this not delete files. The best I'm coming up with is `skip-delete-snapshots`, which is clear that it is referring to snapshot operations. I don't think it would be misinterpreted if it were used in a batch context since it wouldn't make sense to run a select that doesn't do anything if the latest snapshot is a delete.


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