This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new dbaa27ebbe3 [fix](agg) memory leak issue in agg operator (#35037)
(#35055)
dbaa27ebbe3 is described below
commit dbaa27ebbe30aa6e0f17b609c2b63db0818c4a7c
Author: Jerry Hu <mrh...@gmail.com>
AuthorDate: Mon May 20 16:50:50 2024 +0800
[fix](agg) memory leak issue in agg operator (#35037) (#35055)
---
be/src/pipeline/pipeline_x/dependency.cpp | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/be/src/pipeline/pipeline_x/dependency.cpp
b/be/src/pipeline/pipeline_x/dependency.cpp
index ec890dac7f1..ba47f935598 100644
--- a/be/src/pipeline/pipeline_x/dependency.cpp
+++ b/be/src/pipeline/pipeline_x/dependency.cpp
@@ -193,6 +193,12 @@ Status AggSharedState::reset_hash_table() {
}
});
+ if (hash_table.has_null_key_data()) {
+ auto st = _destroy_agg_status(
+ hash_table.template
get_null_key_data<vectorized::AggregateDataPtr>());
+ RETURN_IF_ERROR(st);
+ }
+
aggregate_data_container.reset(new
vectorized::AggregateDataContainer(
sizeof(typename HashTableType::key_type),
((total_size_of_aggregate_states +
align_aggregate_states - 1) /
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
