danielcweeks commented on PR #18329: URL: https://github.com/apache/iceberg/pull/18329#issuecomment-5920048981
My higher-level concern is that we're making an optimization that subtly changes the contract behavior. We bypass permissions checks and the underlying SDK doesn't do that, which is why we see the zero byte request pass through. I don't feel like this would cause problems, but at the same time I feel like it's a narrow optimization case that makes things a little more complicated. It sounds like the original problem was that we produced an invalid object range. Alternatively, we could maintain the full IO contract at just fix that issue. -- 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]
