ygf11 commented on issue #4389: URL: https://github.com/apache/arrow-datafusion/issues/4389#issuecomment-1331669666
> If we can change the pub on: Vec<(column,column)> to option<expr>, we don't need to do the https://github.com/apache/arrow-datafusion/pull/4353 specifically for the expr in the Join plan. Thank you @liukun4515. Avoid adding type coercion specifically for join makes sense to me, but we need split the expression to join keys every time we use it, this is a regular operation in optimizer. Another way to avoid adding the special type coercion, is: * Still use `Vec<(Expr, Expr)>` in `LogicalPlan::Join`. * Combine left and right join keys in `logical_plan.expressions()`. * Split casted equality expression to left and right join keys in `from_plan` when doing type coercion. I'm not sure if this way will change a lot, so need some tries. -- 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]
