RussellSpitzer commented on PR #16939: URL: https://github.com/apache/iceberg/pull/16939#issuecomment-4790187165
I mean this sounds technically correct. But a writer who created such a delete file is probably also violating the spec by writing a delete reference to file it couldn't apply to based on partition boundaries. I assume the reason the check isn't there is because it would be expensive at "addDelete" time to look up the file entry and figure out what it's partition tuple is. -- 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]
