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 a8be47f3ff9 [fix](fs) Close local file writer when downloading via 
broker fs (#34714)
a8be47f3ff9 is described below

commit a8be47f3ff9140ba927e2bbb4029f2783e7feb92
Author: walter <w41te...@gmail.com>
AuthorDate: Sat May 11 19:55:12 2024 +0800

    [fix](fs) Close local file writer when downloading via broker fs (#34714)
---
 be/src/io/fs/broker_file_system.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/io/fs/broker_file_system.cpp 
b/be/src/io/fs/broker_file_system.cpp
index 44582a0ff8d..6096a0540cb 100644
--- a/be/src/io/fs/broker_file_system.cpp
+++ b/be/src/io/fs/broker_file_system.cpp
@@ -416,7 +416,7 @@ Status BrokerFileSystem::download_impl(const Path& 
remote_file, const Path& loca
         RETURN_IF_ERROR(local_writer->append({read_buf.get(), read_len}));
     } // file_handler should be closed before calculating checksum
 
-    return Status::OK();
+    return local_writer->close();
 }
 
 Status BrokerFileSystem::read_file(const TBrokerFD& fd, size_t offset, size_t 
bytes_req,


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

Reply via email to