jayzhan211 commented on PR #14268: URL: https://github.com/apache/datafusion/pull/14268#issuecomment-2632938494
> > @shehabgamin #14440 I come out a flexible version of Signature::CoercibleV2 (temporary name), it can replace `Signature::Coercible`, `Signature::String`, `Signature::Numeric`, `Signature::Exact`, `Signature::Uniform` and `Signature::Comparable`. > > The most difference is that the `implicit coercion` is now defined by the user. This way any change for datafusion doesn't impact downstream projects unlike the existing signature like `Signature::String` or `Signature::Coercible`. > > I probably don't have time to push it forward in the recent days, if you are interested in it you can work on it. > > I think it makes sense to work on this for DataFusion 46! > > > We can implement first version for the functions you mentioned, I believe the change makes more sense > > @jayzhan211 If I am understanding you correctly, Done! I added `AnyNative` and applied it to the functions mentioned. > > CC @alamb @Omega359 I don't quite understand why we are adding more `TypeSignatureClass` here 😕 -- 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
