alamb commented on issue #26257: URL: https://github.com/apache/datafusion/issues/26257#issuecomment-6155013959
> To clarify, the PR was not intended to handle LIMIT + OFFSET overflowing an i64. 😄 The part [@alamb](https://github.com/alamb) mentioned was added to address a review comment. I was focused on addressing the comment and didn't think too much about it. yes, sorry @viirya for calling you out with more context. I just thought that was a good example where well meaning interactions resulted in bunch more complexity than maybe we really needed (it was fixed before merging) -- 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]
