[doris] 03/04: Revert "[fix](agg) need to call 'set_version' in nested functions (#24381)"

[kxiao]

This is an automated email from the ASF dual-hosted git repository.

kxiao pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 336d181e5c45ec17e1209cd5c57e0b8eab895313
Author: Kang <kxiao.ti...@gmail.com>
AuthorDate: Sat Sep 16 23:20:26 2023 +0800

    Revert "[fix](agg) need to call 'set_version' in nested functions (#24381)"
    
    This reverts commit 2ac1c4acd4698360df3a1bbd699ff98a9be4ebb6.
---
 be/src/vec/aggregate_functions/aggregate_function_null.h        | 5 -----
 be/src/vec/aggregate_functions/aggregate_function_state_merge.h | 5 -----
 be/src/vec/aggregate_functions/aggregate_function_state_union.h | 5 -----
 3 files changed, 15 deletions(-)

diff --git a/be/src/vec/aggregate_functions/aggregate_function_null.h 
b/be/src/vec/aggregate_functions/aggregate_function_null.h
index becb06f7cf..a0bb79dfc0 100644
--- a/be/src/vec/aggregate_functions/aggregate_function_null.h
+++ b/be/src/vec/aggregate_functions/aggregate_function_null.h
@@ -81,11 +81,6 @@ public:
         }
     }
 
-    void set_version(const int version_) override {
-        IAggregateFunctionHelper<Derived>::set_version(version_);
-        nested_function->set_version(version_);
-    }
-
     String get_name() const override {
         /// This is just a wrapper. The function for Nullable arguments is 
named the same as the nested function itself.
         return nested_function->get_name();
diff --git a/be/src/vec/aggregate_functions/aggregate_function_state_merge.h 
b/be/src/vec/aggregate_functions/aggregate_function_state_merge.h
index 052f4e6e85..7afd79a1c8 100644
--- a/be/src/vec/aggregate_functions/aggregate_function_state_merge.h
+++ b/be/src/vec/aggregate_functions/aggregate_function_state_merge.h
@@ -38,11 +38,6 @@ public:
         return std::make_shared<AggregateStateMerge>(function, argument_types, 
return_type);
     }
 
-    void set_version(const int version_) override {
-        IAggregateFunctionHelper::set_version(version_);
-        _function->set_version(version_);
-    }
-
     String get_name() const override { return _function->get_name() + 
AGG_MERGE_SUFFIX; }
 
     DataTypePtr get_return_type() const override { return 
_function->get_return_type(); }
diff --git a/be/src/vec/aggregate_functions/aggregate_function_state_union.h 
b/be/src/vec/aggregate_functions/aggregate_function_state_union.h
index b2ff81b168..2b2748828e 100644
--- a/be/src/vec/aggregate_functions/aggregate_function_state_union.h
+++ b/be/src/vec/aggregate_functions/aggregate_function_state_union.h
@@ -42,11 +42,6 @@ public:
         return std::make_shared<AggregateStateUnion>(function, argument_types, 
return_type);
     }
 
-    void set_version(const int version_) override {
-        IAggregateFunctionHelper::set_version(version_);
-        _function->set_version(version_);
-    }
-
     void create(AggregateDataPtr __restrict place) const override { 
_function->create(place); }
 
     String get_name() const override { return _function->get_name() + 
AGG_UNION_SUFFIX; }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org