Skip to content

[feature](new-scan) enable new scan by default#13760

Closed
morningman wants to merge 2 commits intoapache:masterfrom
morningman:enable_new_scan
Closed

[feature](new-scan) enable new scan by default#13760
morningman wants to merge 2 commits intoapache:masterfrom
morningman:enable_new_scan

Conversation

@morningman
Copy link
Contributor

@morningman morningman commented Oct 28, 2022

Proposed changes

Issue Number: close #xxx

Problem summary

Enable new scan node by default.

Checklist(Required)

  1. Does it affect the original behavior:
    • Yes: new scan node only affect vec-engine
    • 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...

@hello-stephen
Copy link
Contributor

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 39.06 seconds
load time: 705 seconds
storage size: 17154831960 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028084324_clickbench_pr_35239.html

@hello-stephen
Copy link
Contributor

hello-stephen commented Oct 28, 2022

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 37.78 seconds
load time: 684 seconds
storage size: 17181336909 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221103092059_clickbench_pr_39166.html

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants