[fix](compatibility) should enable windown funnel mode from 2.0 (#32284)
This commit is contained in:
@ -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.
|
||||
|
||||
@ -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"; }
|
||||
|
||||
Reference in New Issue
Block a user