Acfboy commented on PR #20306: URL: https://github.com/apache/datafusion/pull/20306#issuecomment-3900465808
> i.e., `coerce_arguments_for_signature()` -> `fields_with_udf()` is _already_ checking for functions that require arguments but that are invoked with no arguments, but this is effectively dead code because we exit early before we get there. If we just remove the early-exit then the behavior seems right to me. What do you think? It seems this only works for cases where UDFs are called without arguments. However, there are other cases failing due to a lack of type-checking in IS NULL. For example, `SELECT * FROM (SELECT 1) WHERE ((1.0 << 2.0) IS NULL);` is expected to report that we can't run float << float, but it actually runs without error after constant folding by the optimizer. -- 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]
