neilconway commented on PR #20306:
URL: https://github.com/apache/datafusion/pull/20306#issuecomment-3904739530

   Proposal:
   
   1. Remove the early return from `coerce_arguments_for_signature()`, because 
that seems wrong in any case. (I'm happy to send a PR for this or you can, 
@Acfboy -- lmk what you prefer)
   2. Fix #20355 and #20342 separately
   
   My understanding is that if we do both of these, we don't need the changes 
to `TypeCoercionRewriter` proposed in this PR -- is that correct?


-- 
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]

Reply via email to