WangGuangxin opened a new pull request, #8965: URL: https://github.com/apache/incubator-gluten/pull/8965
## What changes were proposed in this pull request? Currently, BroadcastNestedLoopJoin doesn't support full outer join. That's because in velox/presto, it supposed that the probe side and build side(broadcast side) are all in one worker, so that it can output mismatch records from build side. But for spark, the probe side are distributed across different executors, so that velox cann't recognize the mismatch records from build side. But if the join condition is empty (which is usually the case where BroadcastedNestedLoopJoin take effect), there is no need to output mismatch records. (Fixes: \#8964) ## How was this patch tested? UT and manually -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
