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 1525e673294 [fix](agg) memory leak issue in agg operator (#35037)
1525e673294 is described below

commit 1525e6732940f0bc6c8ae07d8c51864c3882cc1f
Author: Jerry Hu <mrh...@gmail.com>
AuthorDate: Sun May 19 00:22:43 2024 +0800

    [fix](agg) memory leak issue in agg operator (#35037)
---
 be/src/pipeline/dependency.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/be/src/pipeline/dependency.cpp b/be/src/pipeline/dependency.cpp
index 7b25abc5c61..80587af0c8a 100644
--- a/be/src/pipeline/dependency.cpp
+++ b/be/src/pipeline/dependency.cpp
@@ -184,6 +184,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

Reply via email to