Skip to content

[fix](Nereids) join order is not right after sql parsing (#28721) - #29194

Merged
morrySnow merged 1 commit into
apache:branch-2.0from
morrySnow:2.0_28721
Dec 28, 2023
Merged

[fix](Nereids) join order is not right after sql parsing (#28721)#29194
morrySnow merged 1 commit into
apache:branch-2.0from
morrySnow:2.0_28721

Conversation

@morrySnow

@morrySnowmorrySnow commented Dec 28, 2023

Copy link
Copy Markdown
Contributor

pick from master
PR #28721
commit e51e94f

for sql

t1, t2 join t3

we should generate plan like:

t1 join (t2 join t3)

but we generate:

(t1 join t2) join t3

to follow legancy planner.

Proposed changes

Issue Number: close #xxx

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

pick from master
PR# 28721
commit e51e94f
for sql
```
t1, t2 join t3
```
we should generate plan like:
```
t1 join (t2 join t3)
```
but we generate:
```
(t1 join t2) join t3
```
to follow legancy planner.
@morrySnow

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

Copy link
Copy Markdown

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.39 seconds
stream load tsv: 563 seconds loaded 74807831229 Bytes, about 126 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 33 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size: 17162126340 Bytes

@morrySnow
morrySnow merged commit 3827209 into apache:branch-2.0Dec 28, 2023
@morrySnow
morrySnow deleted the 2.0_28721 branch December 28, 2023 07:45
Sign up for freeto 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.

2 participants

@morrySnow@doris-robot