Skip to content

Backport to 20.8 - Disable S3 requests processing during context shutdown to speed up termination process #14496#14858

Merged
alesapin merged 3 commits intoClickHouse:20.8from
Jokser:disk-s3-shutdown-20.8
Sep 16, 2020
Merged

Backport to 20.8 - Disable S3 requests processing during context shutdown to speed up termination process #14496#14858
alesapin merged 3 commits intoClickHouse:20.8from
Jokser:disk-s3-shutdown-20.8

Conversation

@Jokser
Copy link
Copy Markdown
Contributor

@Jokser Jokser commented Sep 15, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Speed up server shutdown process if there are ongoing S3 requests

Detailed description / Documentation draft:
The shutdown process can be delayed if there are ongoing S3 requests, especially when S3 is unavailable (request is repeated again and again due to retry strategy).
When shutdown is called we can forcibly break all S3 connections and speed up the shutdown process.

(cherry picked from commit 9d11d4d)
@Jokser Jokser changed the title Backport to 20.8 - Disable S3 requests processing during context shutdown to speed up termination process #144966 Backport to 20.8 - Disable S3 requests processing during context shutdown to speed up termination process #14496 Sep 15, 2020
@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Sep 15, 2020
@alesapin
Copy link
Copy Markdown
Member

Test failures caused by bugs in the CI. I'll merge, let's check it in the branch.

@alesapin alesapin merged commit 7f0dc0c into ClickHouse:20.8 Sep 16, 2020
@qoega qoega added the pr-backport Changes, backported to release branch. Do not use manually - automated use only! label Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backport Changes, backported to release branch. Do not use manually - automated use only! pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants