xiedeyantu commented on PR #22274: URL: https://github.com/apache/datafusion/pull/22274#issuecomment-4526777564
> @xiedeyantu Thanks for tightening up the overflow handling here. I found two remaining paths where we can still hit unchecked allocation or Arrow offset construction before returning the intended execution error. @kosiew Thanks for the review! I’ll wait until #22305 is merged before making adjustments to this PR, as I’m concerned about potential merge conflicts. -- 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]
