Niketion commented on issue #22506: URL: https://github.com/apache/datafusion/issues/22506#issuecomment-5395277951
I traced the execution path a bit further. execute_prepared already casts the supplied scalar once to the type stored in PreparedPlan, while replace_params_with_values ignores each placeholder occurrence’s expected type. That makes per-occurrence casting a viable alternative to merging everything through comparison_coercion, which fixes Utf8/Utf8View but still coerces string/numeric cases to the numeric type. Would you prefer the minimal comparison_coercion fix, or per-occurrence casting? I can add tests for both occurrence orders, string/numeric cases, metadata, and untyped placeholders. @RatulDawar, are you still working on this, or would you be happy for me to take it over? -- 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]
