Skip to content

[fix](Nereids) Add exchange node check between local and global agg#12913

Merged
morrySnow merged 3 commits intoapache:masterfrom
yinzhijian:dev.fix_distinct
Oct 25, 2022
Merged

[fix](Nereids) Add exchange node check between local and global agg#12913
morrySnow merged 3 commits intoapache:masterfrom
yinzhijian:dev.fix_distinct

Conversation

@yinzhijian
Copy link
Contributor

Proposed changes

Issue Number: close #xxx

Problem summary

table schema

CREATE TABLE t1 (
k1 int(11) NULL,
v1 int(11) NULL
) ENGINE=OLAP
DUPLICATE KEY(k1, v1)
COMMENT 'OLAP'
DISTRIBUTED BY HASH(k1) BUCKETS 3

query

select k1,count(distinct v1+1) from t1 group by k1;

error

java.lang.ClassCastException: org.apache.doris.planner.OlapScanNode cannot be cast to org.apache.doris.planner.AggregationNode

Checklist(Required)

  1. Does it affect the original behavior:
    • Yes
    • No
    • I don't know
  2. Has unit tests been added:
    • Yes
    • No
    • No Need
  3. Has document been added or modified:
    • Yes
    • No
    • No Need
  4. Does it need to update dependencies:
    • Yes
    • No
  5. Are there any changes that cannot be rolled back:
    • Yes (If Yes, please explain WHY)
    • No

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...

@morrySnow morrySnow merged commit f209b7a into apache:master Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants

Comments