liukun4515 commented on PR #5087:
URL: 
https://github.com/apache/arrow-datafusion/pull/5087#issuecomment-1407272597

   why the distribution should be consistent between NLJ and Cross-join?
   When I first implement the NLJ, I didn't consider the `JoinSelection` for 
the NLJ, I think it is an optimization for NLJ.
   I think it is not a good way to change the rule of distribution to fix the 
bug of https://github.com/apache/arrow-datafusion/issues/5022
   
   In many join type, we should have the all data for the left side or the all 
data for the right side.
   
   


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