felixloesing commented on PR #10541: URL: https://github.com/apache/incubator-gluten/pull/10541#issuecomment-3228988171
> @felixloesing Thanks for your fix. Just curious — why does Gluten require this change, while vanilla Spark doesn't? @JkSelf This is also described in the [PR](https://github.com/apache/incubator-gluten/pull/8116) that introduced the `BroadcastMode` to `BuildSideRelation` but let me summarize. Vanilla Spark doesn’t have this because it computes the transform inside BroadcastExchangeExec, so there’s no post‑reuse mismatch to reconcile. In Gluten, if exchange re-use, the buildKeys of ColumnarSubqueryBroadcastExec and the buildKeys of BroadcastExchangeExec may not be the same, so it is difficult to construct the projection of the transform. -- 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]
