-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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.
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
[feature](agg-func) linear_histogram #39546
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TPC-H: Total hot run time: 38361 ms
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add function in fe and add .groovy sql testcase
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
be/src/vec/aggregate_functions/aggregate_function_linear_histogram.h
Outdated
Show resolved
Hide resolved
TPC-DS: Total hot run time: 190978 ms
|
ClickBench: Total hot run time: 30.97 s
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code format is needed, see details in the log:
https://github.com/apache/doris/actions/runs/10446613781/job/28924273137?pr=39546
Tip: you can use the IDE(vscode, clion...) to format the codes automatically.
@superdiaodiao @yiguolei Hello, is this class GenerateFunction used to generate new functions? When I run the test function
Am I using it incorrectly? |
Actually, we have to add FE codes manually rather than generating it, since generating code was out of date and served as a temporary method before. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
Thank you. I add it manually. But when I did the e2e test, I got a strange error. (This error did not occur when I did unit test in the be)
The result of the first execution:
The result of the second execution:
The result of the third execution:
... Only the result of the third execution is correct. I am very confused about this. Why this happens? |
Sorry, I have fixed it. I just forgot to set parameters in merge function~~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
run buildall |
TPC-H: Total hot run time: 37898 ms
|
TPC-DS: Total hot run time: 186595 ms
|
ClickBench: Total hot run time: 31.11 s
|
@superdiaodiao Hello, what 'P0 Regression (Doris Regression) — TeamCity build failed' means? I can not see the details. |
You can click the Details: |
regression-test/suites/nereids_function_p0/agg_function/agg.groovy
Outdated
Show resolved
Hide resolved
run buildall |
TPC-H: Total hot run time: 37709 ms
|
TPC-DS: Total hot run time: 191420 ms
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 41144 ms
|
run cloud_p0 |
run performance |
TPC-H: Total hot run time: 41731 ms
|
TPC-DS: Total hot run time: 195343 ms
|
ClickBench: Total hot run time: 32.56 s
|
# Versions - [x] dev - [x] 3.0 - [ ] 2.1 - [ ] 2.0 # Languages - [x] Chinese - [x] English # Function PR [linear_histogram](apache/doris#39546) in doris
91ed344
run buildall |
TeamCity be ut coverage result: |
run buildall |
TeamCity be ut coverage result: |
run cloud_p0 |
PR approved by at least one committer and no changes requested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
add aggregate function: `linear_histogram(expr, DOUBLE interval[, DOUBLE offset)` The linear_histogram function is used to describe the distribution of the data, It uses an "equal width" bucking strategy, and divides the data into buckets according to the value of the data.
Proposed changes
add aggregate function: linear_histogram
LINEAR_HISTOGRAM
Description
linear_histogram(expr, DOUBLE interval[, DOUBLE offset)
The linear_histogram function is used to describe the distribution of the data, It uses an "equal width" bucking strategy, and divides the data into buckets according to the value of the data.
Parameter description:
Example
Field description:
Doc PR
linear_histogram in doris-website