You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2023/04/28 10:41:49 UTC
[doris] branch master updated: [fix](memory) Fix Aggregation null key memory leak due to incorrect aggfunc destroy #19201
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 a324ee794c [fix](memory) Fix Aggregation null key memory leak due to incorrect aggfunc destroy #19201
a324ee794c is described below
commit a324ee794ce2360e10a9ebb90ae0acf55017aa6b
Author: Xinyi Zou <zo...@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 066b1d98da..e998b2127d 100644
--- a/be/src/vec/exec/vaggregation_node.cpp
+++ b/be/src/vec/exec/vaggregation_node.cpp
@@ -1778,6 +1778,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-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org