alamb opened a new issue, #14291: URL: https://github.com/apache/datafusion/issues/14291
### Describe the bug There is concern changes in this PR will cause regression (as it may convert some numbers to Decimal128 rather than more efficient Integer) - https://github.com/apache/datafusion/pull/14223 However, it is not clear (to me at least) exactly the implications of the change @berkaysynnada has found a potential performance regression, as described here: https://github.com/apache/datafusion/pull/14223#issuecomment-2612423408 cc @nuno-faria @jonahgao and @berkaysynnada and @ozankabak ### To Reproduce _No response_ ### Expected behavior I want to make sure that this issue is resolved before we release datafusion 45.0.0, so filing this ticket so we don't forget Examples of ways to resolve the ticket: - Decide the change was fine - Revert the changes - Others ### Additional context _No response_ -- 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: github-unsubscr...@datafusion.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
