daksha121 commented on pull request #2752: URL: https://github.com/apache/iceberg/pull/2752#issuecomment-877470369
> 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. Thanks @rdblue, that makes sense. Can you help understand why we wouldn't want to put the word streaming in like `streaming-skip-deletes`? We could perhaps eliminate the risk of misinterpretation. Maybe I'm missing something here -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
