comphead commented on PR #3168:
URL: 
https://github.com/apache/arrow-datafusion/pull/3168#issuecomment-1216068817

   > I think a good approach might be to have the cast part of the logical 
planner, so the join expression becomes in the form `col1::TEXT = col2 AND 
col2::`. That shouldn't be really very join specific
   > 
   > I believe we should already have most of the logic there for casting to a 
common type for expressions.
   
   Yes, that is something we discussed with @alamb, that other sql engines have 
cast as part of planner if join arms are not the same datatype but still 
castable.
   
   So what your idea @Dandandan, to try move cast into the planner?


-- 
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]

Reply via email to