zhanglistar opened a new pull request, #11466:
URL: https://github.com/apache/incubator-gluten/pull/11466

   ## What changes are proposed in this pull request?
   - Re-enable base validation for CH BroadcastNestedLoopJoin to reject invalid 
build sides (e.g., LeftOuter + BuildLeft).
   - Add a regression test for left join with an existing but empty partition 
on the build side.
   Fix [#11451](https://github.com/apache/incubator-gluten/issues/11451).
   
   ## Why is this change needed?
   Gluten CH was accepting LeftOuter + BuildLeft in BNLJ, which violates Spark 
join semantics. When the right side is empty, the streamed side becomes empty 
and drops left rows, producing incorrect results compared to vanilla Spark.
   
   ## How was this patch tested?
   - Added unit test in `GlutenClickHouseJoinSuite`.
   


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

Reply via email to