[enhancement](aggregate-function) enhance aggregate funtion collect and add group_array aliases#15339
Conversation
|
clang-tidy review says "All clean, LGTM! 👍" |
d96ac9f to
85e5e4d
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
a677fab to
5ffe8fe
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
5ffe8fe to
5f7af71
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity pipeline, clickbench performance test result: |
5f7af71 to
4c8c13f
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
4c8c13f to
186bc23
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
186bc23 to
262453a
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
262453a to
67817fb
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
67817fb to
99165dc
Compare
4c74780 to
49ada0f
Compare
4a79b47 to
f7fd5f8
Compare
|
run buildall |
|
run p0 |
|
run p1 |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
Hi, Does the case in this pr need to add some |
It seems so, I will add |
…nd add group_array aliases (apache#15339) Enhance aggregate function `collect_set` and `collect_list` to support optional `max_size` param, which enables to limit the number of elements in result array.
Proposed changes
Issue Number: close #13982
Problem summary
Enhance aggregate function
collect_setandcollect_listto support optionalmax_sizeparam, which enables to limit the number of elements in result array.Checklist(Required)
Further comments
Regression tests will be added soon.
If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...