Skip to content

[GLUTEN-6981][CH]Not supported operator TakeOrderedAndProjectExecTransformer for BroadcastRelation#6982

Merged
baibaichen merged 1 commit into
apache:mainfrom
loudongfeng:takeOrderedAndProjectExecTransformer
Aug 23, 2024
Merged

[GLUTEN-6981][CH]Not supported operator TakeOrderedAndProjectExecTransformer for BroadcastRelation#6982
baibaichen merged 1 commit into
apache:mainfrom
loudongfeng:takeOrderedAndProjectExecTransformer

Conversation

@loudongfeng
Copy link
Copy Markdown
Contributor

@loudongfeng loudongfeng commented Aug 22, 2024

What changes were proposed in this pull request?

In some cases, the child node of broadcast is TakeOrderedAndProjectExecTransformer, which is not supported by now.
Add TakeOrderedAndProjectExecTransformer node case support for broadcast .
please refer to #6981 for more details
Fixes: #6981

How was this patch tested?

by UT

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 22, 2024

#6981

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI

@loudongfeng loudongfeng changed the title [GLUTEN-1632][CH]Not supported operator TakeOrderedAndProjectExecTran… [GLUTEN-6981][CH]Not supported operator TakeOrderedAndProjectExecTran… Aug 23, 2024
@github-actions
Copy link
Copy Markdown

#6981

@loudongfeng loudongfeng force-pushed the takeOrderedAndProjectExecTransformer branch from 290843e to 09dd24c Compare August 23, 2024 01:27
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI

@baibaichen baibaichen changed the title [GLUTEN-6981][CH]Not supported operator TakeOrderedAndProjectExecTran… [GLUTEN-6981][CH]Not supported operator TakeOrderedAndProjectExecTransformer for BroadcastRelation Aug 23, 2024
@baibaichen baibaichen force-pushed the takeOrderedAndProjectExecTransformer branch from 09dd24c to 2973713 Compare August 23, 2024 04:46
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI

@baibaichen
Copy link
Copy Markdown
Contributor

@loudongfeng let me rerun ci, and would you please and some comments for what to do?

@loudongfeng
Copy link
Copy Markdown
Contributor Author

@loudongfeng let me rerun ci, and would you please and some comments for what to do?

Done. Please check.

Copy link
Copy Markdown
Contributor

@baibaichen baibaichen left a comment

Choose a reason for hiding this comment

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

LGTM

@baibaichen
Copy link
Copy Markdown
Contributor

Thanks! @loudongfeng

@baibaichen baibaichen merged commit 1844761 into apache:main Aug 23, 2024
sharkdtu pushed a commit to sharkdtu/gluten that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CH] Not supported operator TakeOrderedAndProjectExecTransformer for BroadcastRelation

2 participants