Amogh-2404 commented on PR #24668: URL: https://github.com/apache/datafusion/pull/24668#issuecomment-5414560581
Addressed both review comments. While checking the retained expression through the physical planner, I also found two signed-minimum paths: interval analysis could fail planning, and wrapping integer negation could incorrectly satisfy an ordering requirement. Overflowing bounds now fall back conservatively, and ordering is dropped when `MIN` may be present. The filter, plan, and result regressions are covered in SLTs. All targeted tests, full Clippy, the repository lint suite, and the extended workspace suite pass. -- 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]
