Skip to content

Commit

Permalink
[Bug](compatibility) fix window funnel function coredump when upgrade (
Browse files Browse the repository at this point in the history
…#39646)

## Proposed changes
this PR #39270 have change the agg
of window funnel
and max_be_exec_version is update to 5, in order to compatibility of the
agg function when upgrade.

<!--Describe your changes.-->
  • Loading branch information
zhangstar333 authored Aug 21, 2024
1 parent 7c3c5c6 commit 75eded0
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,9 @@ class AggregateFunctionSimpleFactory {
AggregateFunctions aggregate_functions;
AggregateFunctions nullable_aggregate_functions;
std::unordered_map<std::string, std::string> function_alias;
/// @TEMPORARY: for be_exec_version=2
/// in order to solve agg of sum/count is not compatibility during the upgrade process
constexpr static int AGG_FUNCTION_NEW = 2;
/// @TEMPORARY: for be_exec_version=5
/// in order to solve agg function is not compatibility during the upgrade process
constexpr static int AGG_FUNCTION_NEW = 5;
/// @TEMPORARY: for be_exec_version < AGG_FUNCTION_NEW. replace function to old version.
std::unordered_map<std::string, std::string> function_to_replace;

Expand Down

0 comments on commit 75eded0

Please sign in to comment.