You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/07/09 10:00:11 UTC

[GitHub] [arrow-datafusion] xudong963 opened a new issue, #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

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

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   Migrate the cross join -> inner join optimization from the planner to the optimizer, and make it as an optimization rule.
   
   Related code: https://github.com/apache/arrow-datafusion/blob/master/datafusion/sql/src/planner.rs#L770
   
   **Additional context**
   `cross join -> inner join optimization`: https://github.com/apache/arrow-datafusion/pull/1566


-- 
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 #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
alamb closed issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer
URL: https://github.com/apache/arrow-datafusion/issues/2859


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


[GitHub] [arrow-datafusion] xudong963 commented on issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
xudong963 commented on issue #2859:
URL: https://github.com/apache/arrow-datafusion/issues/2859#issuecomment-1248060686

   @sadilet  Hi, https://github.com/apache/arrow-datafusion/pull/3482 there is a related PR.


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


[GitHub] [arrow-datafusion] sadilet commented on issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
sadilet commented on issue #2859:
URL: https://github.com/apache/arrow-datafusion/issues/2859#issuecomment-1228737742

   @Veeupup Hey, how is it coming along? @xudong963 could I also try to fix this issue?


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


[GitHub] [arrow-datafusion] Veeupup commented on issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
Veeupup commented on issue #2859:
URL: https://github.com/apache/arrow-datafusion/issues/2859#issuecomment-1193097632

   hi, @xudong963 I'm interested in trying this, may I try this?


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


[GitHub] [arrow-datafusion] xudong963 commented on issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
xudong963 commented on issue #2859:
URL: https://github.com/apache/arrow-datafusion/issues/2859#issuecomment-1229114770

   @sadilet  Thanks!


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


[GitHub] [arrow-datafusion] Veeupup commented on issue #2859: Migrate the `cross join -> inner join optimization` from the planner to the optimizer

Posted by GitBox <gi...@apache.org>.
Veeupup commented on issue #2859:
URL: https://github.com/apache/arrow-datafusion/issues/2859#issuecomment-1229095959

   sorry, I have been busy with some other stuff these times and forgot it...
   @sadilet feel free to take it!


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