Skip to content

[fix](planner) Fix wrong results when select from view which has with clause - #14747

Merged
morrySnow merged 1 commit into
apache:masterfrom
Kikyou1997:fix/with_as
Dec 2, 2022
Merged

[fix](planner) Fix wrong results when select from view which has with clause#14747
morrySnow merged 1 commit into
apache:masterfrom
Kikyou1997:fix/with_as

Conversation

@Kikyou1997

@Kikyou1997Kikyou1997 commented Dec 1, 2022

Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close #xxx

Problem summary

Describe your changes.

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

@github-actionsgithub-actionsBot added area/planner Issues or PRs related to the query planner kind/test labels Dec 1, 2022
@hello-stephen

hello-stephen commented Dec 1, 2022

Copy link
Copy Markdown
Contributor

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.76 seconds
load time: 428 seconds
storage size: 17123356256 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221202044553_clickbench_pr_56600.html

@yiguoleiyiguolei added dev/1.2.1 usercase Important user case type label labels Dec 2, 2022
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actionsgithub-actionsBot added approved Indicates a PR has been approved by one committer. reviewed labels Dec 2, 2022
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.area/plannerIssues or PRs related to the query plannerdev/merge-1.1.2kind/testreviewedusercaseImportant user case type label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Kikyou1997@hello-stephen@morrySnow@morningman@yiguolei