[GitHub] [arrow-datafusion] tustvold commented on pull request #6221: refactor: separate get_result_type from `coerce_type`

["tustvold (via GitHub)" <gi...@apache.org>]

tustvold commented on PR #6221:
URL: https://github.com/apache/arrow-datafusion/pull/6221#issuecomment-1609113819

   One slightly unfortunate quirk of this separation is it isn't clear if the inputs to get_result_type are before or after type coercion. This has non-trivial implications for decimal arithmetic, in particular for division, as type coercion is not idempotent. I'm still thinking about how to handle this better, but thought I would mention it in case you've already given this some thought


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org