alamb commented on PR #23211: URL: https://github.com/apache/datafusion/pull/23211#issuecomment-4845518753
> #21883 introduced string_concat operand coercion specifically for `FixedSizeBinary` to `Binary`. This causes DataFusion to not use the `FixedSizeBinary` code path that's now enabled by [apache/arrow-rs#10222](https://github.com/apache/arrow-rs/pull/10222). Is there a reason to keep this or would it be preferable to make `||` return `FixedSizeBinary(l + r)`? I think we'll have to ask @theirix -- 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]
