This is an automated email from the ASF dual-hosted git repository.
dataroaring 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 a8f16add35c [fix](cloud) Prevent S3 error log name collisions in
multiple instances (#51356)
a8f16add35c is described below
commit a8f16add35c234ee2d924d2826c4d69973f5e2b7
Author: Xin Liao <[email protected]>
AuthorDate: Fri May 30 09:54:26 2025 +0800
[fix](cloud) Prevent S3 error log name collisions in multiple instances
(#51356)
Check failed: bytes_appended == head_res.file_size (261 vs. 87) failed
to check size after upload
Multiple instances writing error logs with the same object name causes
S3 writer size check failures since different instances could overwrite
each other's error logs.
---
be/src/runtime/runtime_state.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/be/src/runtime/runtime_state.cpp b/be/src/runtime/runtime_state.cpp
index b9e96ad0892..86536a4bce6 100644
--- a/be/src/runtime/runtime_state.cpp
+++ b/be/src/runtime/runtime_state.cpp
@@ -307,7 +307,7 @@ Status RuntimeState::create_error_log_file() {
//
https://dev.mysql.com/doc/dev/mysql-server/latest/page_protocol_basic_err_packet.html
// shorten the path as much as possible to prevent the length of
the presigned URL from
// exceeding the MySQL error packet size limit
- ss << "error_log/" << std::hex << _query_id.hi;
+ ss << "error_log/" << std::hex << _fragment_instance_id.lo;
_s3_error_log_file_path = ss.str();
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
