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

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


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new e1a1b7f9b32 branch-3.0: [fix](beut) Fix `s3_file_writer_test` maybe 
memory leak #47901 (#47952)
e1a1b7f9b32 is described below

commit e1a1b7f9b322353d2e78f545004c12cfaa76f1d8
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Feb 17 10:46:55 2025 +0800

    branch-3.0: [fix](beut) Fix `s3_file_writer_test` maybe memory leak #47901 
(#47952)
    
    Cherry-picked from #47901
    
    Co-authored-by: Lei Zhang <[email protected]>
---
 be/test/io/fs/s3_file_writer_test.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/be/test/io/fs/s3_file_writer_test.cpp 
b/be/test/io/fs/s3_file_writer_test.cpp
index 010ef7e1c1e..79af666f928 100644
--- a/be/test/io/fs/s3_file_writer_test.cpp
+++ b/be/test/io/fs/s3_file_writer_test.cpp
@@ -1121,7 +1121,7 @@ public:
     ObjectStorageUploadResponse upload_part(const ObjectStoragePathOptions& 
opts,
                                             std::string_view stream, int 
part_num) override {
         upload_part_count++;
-        upload_part_params.push_back({opts, std::string(stream), part_num});
+        // upload_part_params.push_back({opts, std::string(stream), part_num});
         last_opts = opts;
         last_stream = std::string(stream);
         last_part_num = part_num;
@@ -1232,7 +1232,7 @@ public:
     // Vectors to store parameters from each call
     std::vector<ObjectStoragePathOptions> create_multipart_params;
     std::vector<std::pair<ObjectStoragePathOptions, std::string>> 
put_object_params;
-    std::vector<UploadPartParams> upload_part_params;
+    // std::vector<UploadPartParams> upload_part_params;
     std::vector<CompleteMultipartParams> complete_multipart_params;
     std::map<std::string, std::string> objects;
     std::map<std::string, std::string> complete;
@@ -1256,7 +1256,7 @@ public:
 
         create_multipart_params.clear();
         put_object_params.clear();
-        upload_part_params.clear();
+        // upload_part_params.clear();
         complete_multipart_params.clear();
         objects.clear();
         complete.clear();


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to