Skip to content

Slow down S3 client after network error#80035

Merged
vitlibar merged 3 commits intoClickHouse:masterfrom
vitlibar:s3-slow-down-after-network-error
May 14, 2025
Merged

Slow down S3 client after network error#80035
vitlibar merged 3 commits intoClickHouse:masterfrom
vitlibar:s3-slow-down-after-network-error

Conversation

@vitlibar
Copy link
Copy Markdown
Member

@vitlibar vitlibar commented May 9, 2025

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Add setting s3_slow_all_threads_after_network_error.

When set to true (default) than all threads executing s3 requests to the same endpoint get slow down for a while after one s3 request fails with a retryable network error. This is the new default.

When set to false than each thread executing s3 request uses an independent set of backoffs on network errors. This is the old behavior.

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented May 9, 2025

Workflow [PR], commit [13a8c91]

@clickhouse-gh clickhouse-gh bot added the pr-improvement Pull request with some product improvements label May 9, 2025
@vitlibar vitlibar force-pushed the s3-slow-down-after-network-error branch 2 times, most recently from e2c89c4 to b1d2743 Compare May 9, 2025 21:39
@mstetsyuk mstetsyuk self-assigned this May 10, 2025
@vitlibar vitlibar force-pushed the s3-slow-down-after-network-error branch from 21fb53a to 13a8c91 Compare May 12, 2025 14:18
@vitlibar vitlibar enabled auto-merge May 12, 2025 20:21
@vitlibar
Copy link
Copy Markdown
Member Author

CI failures are unrelated:

@vitlibar vitlibar added this pull request to the merge queue May 14, 2025
Merged via the queue into ClickHouse:master with commit 47a3bf1 May 14, 2025
117 of 122 checks passed
@vitlibar vitlibar deleted the s3-slow-down-after-network-error branch May 14, 2025 11:25
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 14, 2025
baibaichen pushed a commit to Kyligence/gluten that referenced this pull request May 15, 2025
baibaichen pushed a commit to apache/gluten that referenced this pull request May 15, 2025
* [GLUTEN-1632][CH]Daily Update Clickhouse Version (20250515)

* Fix build due to ClickHouse/ClickHouse#80035

* Fix build due to ClickHouse/ClickHouse#80050

---------

Co-authored-by: kyligence-git <gluten@kyligence.io>
Co-authored-by: Chang chen <chenchang@apache.com>
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-backports-created-cloud deprecated label, NOOP label Jun 13, 2025
@robot-clickhouse robot-clickhouse added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created-cloud deprecated label, NOOP pr-improvement Pull request with some product improvements pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants