kumarUjjawal commented on issue #12725:
URL: https://github.com/apache/datafusion/issues/12725#issuecomment-3695450998
I was looking at min_max and found that min/max could be a valid use case
for `Signature::user_defined` for now:
When I changed from `Signature::user_defined` to `Signature::any(1)`:
- Basic min/max operations work fine
- Dictionary-typed columns fail (wrong results + panics)
The `coerce_types()` method is only invoked when the signature is
`TypeSignature::UserDefined`. With `Signature::any(1)`, our `coerce_types()`
for dictionary unwrapping is never called, causing dictionary values to be
passed directly to accumulators that can't handle them.
@Jefffrey did you get the time to look at it?
--
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]
