[Improve](sort) avoid too may tmp vectors for get_columns - #27734

Merged

xiaokang merged 2 commits into

apache:masterfrom

eldenmoon:master

Nov 30, 2023

eldenmoon commented Nov 29, 2023

Member

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

eldenmoon commented Nov 29, 2023

MemberAuthor

run buildall

eldenmoon added the dev/2.0.4 label

github-actionsBot commented Nov 29, 2023

Contributor

clang-tidy review says "All clean, LGTM! 👍"

eldenmoon commented Nov 29, 2023

MemberAuthor

run buildall

github-actionsBot commented Nov 29, 2023

Contributor

clang-tidy review says "All clean, LGTM! 👍"

[Improve](sort) avoid too may tmp vectors for get_columns

51ae024

eldenmoon force-pushed the master branch 2 times, most recently from 8a8f07a to 51ae024Compare

November 29, 2023 03:23

github-actionsBot commented Nov 29, 2023

Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment

github-actionsBot commented Nov 29, 2023

Contributor

clang-tidy review says "All clean, LGTM! 👍"

Merge branch 'master' into master

e8346e5

eldenmoon commented Nov 29, 2023

MemberAuthor

run buildall

github-actionsBot commented Nov 29, 2023

Contributor

clang-tidy review says "All clean, LGTM! 👍"

doris-robot commented Nov 29, 2023

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 43.44 seconds
stream load tsv: 569 seconds loaded 74807831229 Bytes, about 125 MB/s
stream load json: 18 seconds loaded 2358488459 Bytes, about 124 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select: 28.8 seconds inserted 10000000 Rows, about 347K ops/s
storage size: 17099309585 Bytes

yiguolei approved these changes

View reviewed changes

yiguolei left a comment

Contributor

lgtm

github-actionsBot added the approved label

github-actionsBot commented Nov 29, 2023

Contributor

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

github-actionsBot added the reviewed label

github-actionsBot commented Nov 29, 2023

Contributor

PR approved by anyone and no changes requested.

xiaokang approved these changes

View reviewed changes

xiaokang left a comment

Contributor

LGTM

xiaokang merged commit e9debca into apache:master

wm1581066 added the p0_b label

XuJianxu pushed a commit to XuJianxu/doris that referenced this pull request

[Improve](sort) avoid too may tmp vectors for get_columns (apache#27734)

275ce6b

eldenmoon added a commit to eldenmoon/incubator-doris that referenced this pull request

[Improve](sort) avoid too may tmp vectors for get_columns (apache#27734)

45dc2d3

eldenmoon mentioned this pull request

[Pick](branch-2.0) pick some PRs about point query optimization #28611

Merged

xiaokang pushed a commit that referenced this pull request

[Pick](branch-2.0) pick some PRs about point query optimization (#28611)

c03c92a

* [Fix](point query) fix memleak by increasing `scanReplicaIds` when using prepared statement (#28184)
* [Performance](point query)Opimize partition prune for point query (#28150)
* [Improve](sort) avoid too may tmp vectors for get_columns (#27734)

xiaokang added dev/2.0.4-merged and removed dev/2.0.4 labels

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