This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new e4f0c6032fa [fix](compatibility) should enable windown funnel mode
from 2.0 (#32284)
e4f0c6032fa is described below
commit e4f0c6032fa83ec1d79a907c74e4cf141e43e305
Author: Jerry Hu <[email protected]>
AuthorDate: Sat Mar 16 08:54:33 2024 +0800
[fix](compatibility) should enable windown funnel mode from 2.0 (#32284)
---
be/src/agent/be_exec_version_manager.h | 3 ++-
be/src/vec/aggregate_functions/aggregate_function_window_funnel.h | 4 ++--
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/be/src/agent/be_exec_version_manager.h
b/be/src/agent/be_exec_version_manager.h
index 0290e3bee32..470d78ee981 100644
--- a/be/src/agent/be_exec_version_manager.h
+++ b/be/src/agent/be_exec_version_manager.h
@@ -60,8 +60,9 @@ private:
* e. add repeat_max_num in repeat function
* 3: start from doris 2.0 (by some mistakes)
* a. aggregation function do not serialize bitmap to string.
+ * b. support window funnel mode.
* 4: start from doris 2.1
- * a. support window funnel mode from 2.0
+ * a. ignore this line, window funnel mode should be enabled from 2.0.
* b. array contains/position/countequal function return nullable in less
situations.
* c. cleared old version of Version 2.
* d. unix_timestamp function support timestamp with float for datetimev2,
and change nullable mode.
diff --git a/be/src/vec/aggregate_functions/aggregate_function_window_funnel.h
b/be/src/vec/aggregate_functions/aggregate_function_window_funnel.h
index ed4c9aed059..9356cfd4b68 100644
--- a/be/src/vec/aggregate_functions/aggregate_function_window_funnel.h
+++ b/be/src/vec/aggregate_functions/aggregate_function_window_funnel.h
@@ -270,8 +270,8 @@ public:
void create(AggregateDataPtr __restrict place) const override {
auto data = new (place) WindowFunnelState<DateValueType, NativeType>();
- /// support window funnel mode from 2.1. See
`BeExecVersionManager::max_be_exec_version`
- data->enable_mode = version >= USE_NEW_SERDE;
+ /// support window funnel mode from 2.0. See
`BeExecVersionManager::max_be_exec_version`
+ data->enable_mode = version >= 3;
}
String get_name() const override { return "window_funnel"; }
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
