shehabgamin commented on PR #14268: URL: https://github.com/apache/datafusion/pull/14268#issuecomment-2629239585
### Topic 1 > @findepi I had thought about using Signature:: User-defined everywhere before but was rejected for some reason. As the discussion here, I think we need this to minimize the impact of any changes that is required for datafusion builtun function but not for other projects. Any thoughts to add in? @alamb @jayzhan211 Are we waiting on @findepi to chime in, or should I just go ahead with this approach? ### Topic 2 > It does seem to me that we keep churning / thrashing on coercion (and introducing regressions like the one this PR fixes along with https://github.com/apache/datafusion/issues/14154 and https://github.com/apache/datafusion/issues/14383 > > To me this is a sign we need more work as @jayzhan211 says and I am sure we would appreciate help figuring out what to do / making the code better I am happy to help out here! I think that we all want the same thing, but what's going on is that it's hard for all of us to be fully contextualized with everything so we end up going in circles. @alamb @jayzhan211 Let's bring the conversation back here https://github.com/apache/datafusion/issues/14296 and try to get this topic resolved for DataFusion 46. I would also like to note that I'm always happy to hop on a call or do some sort of live messaging if that's ever desired. -- 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
