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 7263f1e91b9 [chore](config) Add `request_timeout` config for aws
client configuration (#60906)
7263f1e91b9 is described below
commit 7263f1e91b9668bd51a09f7129fb49c5775d859e
Author: Yixuan Wang <[email protected]>
AuthorDate: Mon Mar 9 19:23:31 2026 +0800
[chore](config) Add `request_timeout` config for aws client configuration
(#60906)
### What problem does this PR solve?
- add `aws_client_request_timeout_ms` config with default `30000`
- wire it to `Aws::Client::ClientConfiguration::requestTimeoutMs` in
`s3_util`
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
---
be/src/common/config.cpp | 1 +
be/src/common/config.h | 1 +
be/src/util/s3_util.h | 1 +
3 files changed, 3 insertions(+)
diff --git a/be/src/common/config.cpp b/be/src/common/config.cpp
index b02ca0ba510..8eae149c963 100644
--- a/be/src/common/config.cpp
+++ b/be/src/common/config.cpp
@@ -1477,6 +1477,7 @@ DEFINE_mInt32(max_s3_client_retry, "10");
DEFINE_mInt32(s3_read_base_wait_time_ms, "100");
DEFINE_mInt32(s3_read_max_wait_time_ms, "800");
DEFINE_mBool(enable_s3_object_check_after_upload, "true");
+DEFINE_mInt32(aws_client_request_timeout_ms, "30000");
DEFINE_mBool(enable_s3_rate_limiter, "false");
DEFINE_mInt64(s3_get_bucket_tokens, "1000000000000000000");
diff --git a/be/src/common/config.h b/be/src/common/config.h
index e2e494af8a9..43be7f9e37a 100644
--- a/be/src/common/config.h
+++ b/be/src/common/config.h
@@ -1587,6 +1587,7 @@ DECLARE_mInt32(max_s3_client_retry);
DECLARE_mInt32(s3_read_base_wait_time_ms);
DECLARE_mInt32(s3_read_max_wait_time_ms);
DECLARE_mBool(enable_s3_object_check_after_upload);
+DECLARE_mInt32(aws_client_request_timeout_ms);
// write as inverted index tmp directory
DECLARE_String(tmp_file_dir);
diff --git a/be/src/util/s3_util.h b/be/src/util/s3_util.h
index 8e956bcb2db..5cedee997ee 100644
--- a/be/src/util/s3_util.h
+++ b/be/src/util/s3_util.h
@@ -150,6 +150,7 @@ public:
// So here we use a static instance, and deep copy every time
// to avoid unnecessary operations.
static Aws::Client::ClientConfiguration instance;
+ instance.requestTimeoutMs = config::aws_client_request_timeout_ms;
return instance;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
