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

   I think CrossJoin is still required as a physical join plan implementation, 
because for Cross Join we can have different Partitioning/Shuffle strategies(M 
* N). Of course we can merge all the logic to NestedLoopJoin, but this will 
make the NestedLoopJoin's partitioning logic very complex(which is already 
quite complex).


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