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
[Enhancement] optimize performance of percentile_cont function #14609
[Enhancement] optimize performance of percentile_cont function #14609
Conversation
8d43a3c
to
ee3fc3b
Compare
clang-tidy review says "All clean, LGTM! 👍" |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
run starrocks_admit_test |
run starrocks_admit_test |
run starrocks_admit_test |
@Mergifyio backport branch-2.5 |
@Mergifyio backport branch-2.4 |
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
✅ Backports have been created
|
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
✅ Backports have been created
|
@Mergifyio backport branch-2.2 branch-2.3 |
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
✅ Backports have been created
|
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
(cherry picked from commit 4e3418c) # Conflicts: # be/src/exprs/agg/percentile_cont.h
What type of PR is this:
Which issues of this PR fixes :
Fixes #14608
Problem Summary(Required) :
Replace the out-of-place merge with in-place merge to reduce memory allocation overhead.
Benchmark result:
select percentile_cont(lo_orderdate, 0.5) from lineorder
with SSB( scale=100)64s
, After is37s
Checklist:
Bugfix cherry-pick branch check: