Skip to content

[VL] Remove support for BNLJ full outer join without condition#11021

Merged
zml1206 merged 3 commits intoapache:mainfrom
zml1206:remove_support_blnj_full
Nov 5, 2025
Merged

[VL] Remove support for BNLJ full outer join without condition#11021
zml1206 merged 3 commits intoapache:mainfrom
zml1206:remove_support_blnj_full

Conversation

@zml1206
Copy link
Contributor

@zml1206 zml1206 commented Nov 4, 2025

What changes are proposed in this pull request?

As this discussion, #8965 (comment), we will remove this support.
Also fix #10989

How was this patch tested?

@github-actions github-actions bot added CORE works for Gluten Core VELOX labels Nov 4, 2025
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Run Gluten Clickhouse CI on x86

Copy link
Contributor

@jinchengchenghh jinchengchenghh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much!

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Run Gluten Clickhouse CI on x86

@zml1206
Copy link
Contributor Author

zml1206 commented Nov 5, 2025

Thanks @jinchengchenghh . Merge into main.

@zml1206 zml1206 merged commit 21c0e43 into apache:main Nov 5, 2025
101 of 102 checks passed
@zhouyuan
Copy link
Member

zhouyuan commented Nov 7, 2025

It looks like we should also backport to 1.5
Cc @PHILO-HE

@PHILO-HE
Copy link
Member

It looks like we should also backport to 1.5 Cc @PHILO-HE

@zhouyuan, +1. I just created a GitHub issue for tracking those patches for backporting: #11058, with this patch added to the tracking list.

@zml1206
Copy link
Contributor Author

zml1206 commented Nov 10, 2025

Backport in #11060 @zhouyuan @PHILO-HE

PHILO-HE pushed a commit that referenced this pull request Nov 14, 2025
@zml1206 zml1206 deleted the remove_support_blnj_full branch December 9, 2025 08:11
zml1206 added a commit to zml1206/incubator-gluten that referenced this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CORE works for Gluten Core VELOX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[VL] Full outer join without join condition in BroadcastNestedLoopJoinExecTransformer failed

4 participants