ozankabak commented on PR #14273: URL: https://github.com/apache/datafusion/pull/14273#issuecomment-2613842458
This change makes sense to me. However, I *fully* agree with @alamb on avoiding being trigger happy on partial changes to coercion behavior. Let's follow the 4-step process laid out in his comment before merging. I don't remember the previous change from floats to decimals, but we should understand what happened then to avoid churn. I want to say it was about Spark compatibility, but I might be misremembering. If that is the case, we'd need to choose PostgreSQL compatibility over Spark compatibility (which is in line with project goals). -- 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 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
