Skip to content

[Opt](func) Improve AggFunc Percentile performance #62520

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

Merged
zclllyybb merged 4 commits into from
Apr 24, 2026
Merged

[Opt](func) Improve AggFunc Percentile performance #62520

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
37dff38
[Opt](func) Improve AggFunc Percentile performance
linrrzqqq Apr 15, 2026
a3ccef3
fix
linrrzqqq Apr 17, 2026
f0fb942
fix register
linrrzqqq Apr 18, 2026
ac8c962
fix ut
linrrzqqq Apr 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions be/src/exprs/aggregate/aggregate_function.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ struct AggregateFunctionAttr {
bool is_window_function {false};
bool is_foreach {false};
bool enable_aggregate_function_null_v2 {false};
bool new_version_percentile {false};
std::vector<std::string> column_names;
};

Expand Down
8 changes: 6 additions & 2 deletions be/src/exprs/aggregate/aggregate_function_percentile.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,13 @@ void register_aggregate_function_percentile(AggregateFunctionSimpleFactory& fact
using creator = creator_with_type_list<TYPE_TINYINT, TYPE_SMALLINT, TYPE_INT, TYPE_BIGINT,
TYPE_LARGEINT, TYPE_FLOAT, TYPE_DOUBLE>;
factory.register_function_both("percentile", creator::creator<AggregateFunctionPercentile>);
factory.register_alias("percentile", "percentile_cont");
factory.register_function_both("percentile_array",
creator::creator<AggregateFunctionPercentileArray>);
factory.register_function_both("percentile_v2",
creator::creator<AggregateFunctionPercentileV2>);
factory.register_function_both("percentile_array_v2",
creator::creator<AggregateFunctionPercentileArrayV2>);
factory.register_alias("percentile", "percentile_cont");
}

void register_percentile_approx_old_function(AggregateFunctionSimpleFactory& factory) {
Expand All @@ -87,4 +91,4 @@ void register_aggregate_function_percentile_approx(AggregateFunctionSimpleFactor
register_percentile_approx_old_function(factory);
}

} // namespace doris
} // namespace doris
Loading
Loading