Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-22]Fix w/DPP issue when inside wscg smj both sides are smj #26

Merged
merged 1 commit into from
Jan 11, 2021
Merged

[NSE-22]Fix w/DPP issue when inside wscg smj both sides are smj #26

merged 1 commit into from
Jan 11, 2021

Conversation

xuechendi
Copy link
Collaborator

When DPP enabled, AQE won't work to replace smj with BHJ, so in Q95, there is one smj whose both sides are smj.

To fix this issue,

  1. we add a new strategy to split two wscg when smj are connected by build side.
  2. we also added a support to treat any operator as buildSide operator in smj.
  3. fix a bug when buildSide input is empty.
  4. Add a strategy that If only one side child is smj, then this side will be treated as streamedSide

Fixed: #22

@xuechendi
Support any operator as smj build side in wscg and divide two smjwscg…
5d4df86 
… if they connect in build side

Signed-off-by: Chendi Xue <[email protected]>
@xuechendi xuechendi force-pushed the wip_split_smjwscg_in_buildside branch from 61d83d8 to 5d4df86 Compare January 10, 2021 13:15
@github-actions
Copy link

@github-actions
Copy link

@zhouyuan zhouyuan merged commit 686232e into oap-project:master Jan 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TPC-DS Q95 failed due in columnar wscg
2 participants
@xuechendi @zhouyuan