pepijnve commented on PR #23211: URL: https://github.com/apache/datafusion/pull/23211#issuecomment-4826130843
#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 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)`? -- 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]
