alamb commented on PR #7529:
URL: 
https://github.com/apache/arrow-datafusion/pull/7529#issuecomment-1717993123

   >  because multi join is a Query Graph and both equal conditions and 
non-equal conditions are edges in Graph. But I also think this is a longer-term 
problem. I prepare to do this job in 1-2 weeks.
   
   Representing the joins in a query as a JoinGraph would be very interesting. 
I wonder if you have an idea of where would it be done? 
   
   It could be as a pass prior to the LogicalPlan:
   ```
   sql text --> Sql AST --> JoinGraph --> LogicalPlan
   ```
   
   or you could do it as part of an optimization pass,  like  this:
   
   ```
   sql text --> Sql AST  --> LogicalPlan --(optimize with JoinGraph) --> 
LogicalPlan
   ```
   


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