[doris] 02/03: [fix](memory) Fix Aggregation null key memory leak due to incorrect aggfunc destroy #19201

[morningman]

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

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

commit 9c5f32fce19da9da56f16cf9bc7bc1639a2d523e
Author: Xinyi Zou <zouxiny...@gmail.com>
AuthorDate: Fri Apr 28 18:41:41 2023 +0800

    [fix](memory) Fix Aggregation null key memory leak due to incorrect aggfunc 
destroy #19201
---
 be/src/vec/exec/vaggregation_node.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/be/src/vec/exec/vaggregation_node.cpp 
b/be/src/vec/exec/vaggregation_node.cpp
index 91ddedd22a..ef61be18eb 100644
--- a/be/src/vec/exec/vaggregation_node.cpp
+++ b/be/src/vec/exec/vaggregation_node.cpp
@@ -1407,6 +1407,9 @@ void AggregationNode::_close_with_serialized_key() {
                         mapped = nullptr;
                     }
                 });
+                if (data.has_null_key_data()) {
+                    _destroy_agg_status(data.get_null_key_data());
+                }
             },
             _agg_data->_aggregated_method_variant);
     release_tracker();


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