stevenzwu commented on pull request #3258: URL: https://github.com/apache/iceberg/pull/3258#issuecomment-940300490
@rdblue with the new logic of removing validation in the PR, I agree that we don't need the failed unit test of `testValidateDataFileExist` > We could also add a check to ensure that there are no such files by validating that the set of referenced data files from the positional deletes is a subset of the data files to add. That sounds like a good idea. -- 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]
