AlejandroMorgante commented on PR #18329: URL: https://github.com/apache/iceberg/pull/18329#issuecomment-5920458331
I agree that returning early bypasses the existence and read-permission checks performed by S3. `InputFile.exists()` only checks whether the object exists; it doesn’t verify read permission. I couldn’t find a valid `Range` for requesting zero bytes, and issuing an un-ranged `GetObject` would request the object body just to discard it. I think a zero-length read should return without an S3 request, but I’m open to your thoughts on the expected contract. -- 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]
