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/12/10 12:57:32 UTC

[GitHub] [arrow-datafusion] jackwener commented on pull request #4562: feat: support nested loop join with the initial version

jackwener commented on PR #4562:
URL: https://github.com/apache/arrow-datafusion/pull/4562#issuecomment-1345259469

   > I wonder if it makes sense to also make use of `NestedLoopJoin` for cross joins (with empty filter) and get rid of `CrossJoinExec`, as that should have the same result while avoiding duplicate code. Does this make sense?
   
   Yes, it make sense. I think CrossJoinExec should be a form of `NLJ`


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