houqp edited a comment on pull request #605:
URL: https://github.com/apache/arrow-datafusion/pull/605#issuecomment-873534193


   @alamb reimplemented the logic based on your suggestion in 
https://github.com/apache/arrow-datafusion/pull/605#issuecomment-866862804.
   
   Turns out there are many more edge-cases that need to be handled for using 
join other than wildcard expansion:
   
   * predicate push down on join columns
   * normalize unqualified column expressions that reference join columns with 
qualifiers
   
   I have implemented support for all these edge-cases, but decided to leave 
out the wildcard expansion change as a follow up PR to keep the diff easier to 
review.
   
   UPDATE: filed https://github.com/apache/arrow-datafusion/pull/678 as follow 
up.


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to