[vectorized](udaf) java udaf support with map type #22397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

zhangstar333 merged 5 commits into apache:master from Mryange:udaf-map

Aug 2, 2023

Contributor

Mryange commented Jul 31, 2023

Proposed changes

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


          test

6afd3b1

Mryange marked this pull request as draft

July 31, 2023 03:24

Contributor Author

Mryange commented Jul 31, 2023

run buildall

Contributor

github-actions bot commented Jul 31, 2023

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


          remove some unused

b416a7e

Contributor Author

Mryange commented Jul 31, 2023

run buildall

Contributor

github-actions bot commented Jul 31, 2023

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

Contributor

hello-stephen commented Jul 31, 2023

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.3 seconds
stream load tsv: 507 seconds loaded 74807831229 Bytes, about 140 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: 31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select: 29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size: 17162322284 Bytes


          update

05423e2

Contributor

github-actions bot commented Jul 31, 2023

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


          add case

dc21782

Contributor Author

Mryange commented Jul 31, 2023

run buildall

Contributor

github-actions bot commented Jul 31, 2023

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

Mryange marked this pull request as ready for review

July 31, 2023 11:43

Contributor Author

Mryange commented Jul 31, 2023

run clickbench


          Merge branch 'master' of https://github.com/apache/doris into udaf-map

30595f1

Contributor Author

Mryange commented Jul 31, 2023

run buildall

Contributor

github-actions bot commented Jul 31, 2023

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

Contributor

hello-stephen commented Jul 31, 2023

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.2 seconds
stream load tsv: 510 seconds loaded 74807831229 Bytes, about 139 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select: 29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size: 17162223035 Bytes

zhangstar333 approved these changes

View reviewed changes

github-actions bot added the approved label

Contributor

github-actions bot commented Aug 1, 2023

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

github-actions bot added the reviewed label

Contributor

github-actions bot commented Aug 1, 2023

PR approved by anyone and no changes requested.

HappenLee approved these changes

View reviewed changes

Contributor

HappenLee left a comment

LGTM

zhangstar333 merged commit ddd9085 into apache:master

Mryange mentioned this pull request

[refactor](udaf) refactor call udaf function and support map type in return #22508

Merged

Mryange added a commit to Mryange/doris that referenced this pull request


          [vectorized](udaf) java udaf support with map type (apache#22397)

a7c8068

[vectorized](udaf) java udaf support with map type (apache#22397)
* test
* remove some unused
* update
* add case

Mryange mentioned this pull request

[pick](udaf) pick udaf update #22893

Merged

xiaokang pushed a commit that referenced this pull request


          [pick](udaf) pick udaf update (#22397) (#22508) (#22893)

2c617f1

[vectorized](udaf) java udaf support with map type (#22397)

[refactor](udaf) refactor call udaf function and support map type in return (#22508)

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

Labels

approved reviewed