jackye1995 commented on pull request #1863: URL: https://github.com/apache/iceberg/pull/1863#issuecomment-737625728
> Would it make sense to test for the contents of a file change after put (and not just the length being the same like is currently tested)? What situation does it try to cover in the context of this PR? (I do have more tests for S3FileIO that I plan to port here in another PR, which checks the contents of a file as you describe) ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
