[GitHub] [arrow-datafusion] jackwener opened a new issue, #4176: Reimplement the `eliminate_cross_join`

[GitBox <gi...@apache.org>]

jackwener opened a new issue, #4176:
URL: https://github.com/apache/arrow-datafusion/issues/4176

   **Describe the bug**
   Current implementation of `eliminate_cross_join` is wrong.
   Because it can't rearrange join order.
   
   When original plan is
   ```
   filter: a.id = b.id and a.id = c.id
     crossjoin a  (bc)
       crossjoin b c
   
   it will become 
   
   filter: a.id = b.id and a.id = c.id
     inner a (bc) on a.id = b.id and a.id = c.id
       crossjoin b c
   ```
   
   I will reimplement `eliminate_cross_join` to resolve this problem.
   
   **To Reproduce**
   Steps to reproduce the behavior:
   
   **Expected behavior**
   A clear and concise description of what you expected to happen.
   
   **Additional context**
   Add any other context about the problem here.
   


-- 
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-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow-datafusion] alamb closed issue #4176: Reimplement the `eliminate_cross_join`

[GitBox <gi...@apache.org>]

alamb closed issue #4176: Reimplement the `eliminate_cross_join`
URL: https://github.com/apache/arrow-datafusion/issues/4176


-- 
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-unsubscribe@arrow.apache.org

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