Explicitly set loglevel for AWS API logging options by thevar1able · Pull Request #104837 · ClickHouse/ClickHouse · GitHub
Skip to content

Explicitly set loglevel for AWS API logging options#104837

Merged
thevar1able merged 5 commits into
masterfrom
fix-disabled-aws-logger
May 18, 2026
Merged

Explicitly set loglevel for AWS API logging options#104837
thevar1able merged 5 commits into
masterfrom
fix-disabled-aws-logger

Conversation

@thevar1able

@thevar1able thevar1able commented May 13, 2026

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

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

Fix AWS logger being disabled after 0e8ad4355c9d.

Version info

  • Merged into: 26.5.1.745

@thevar1able thevar1able marked this pull request as ready for review May 13, 2026 12:34
@antonio2368 antonio2368 self-assigned this May 13, 2026
@clickhouse-gh

clickhouse-gh Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-bugfix Pull request with bugfix, not backported by default label May 13, 2026
@thevar1able

Copy link
Copy Markdown
Member Author

Wow, TSan is not happy.

Comment thread src/IO/S3/Client.cpp Outdated
Comment thread src/IO/S3/Client.cpp
@clickhouse-gh

clickhouse-gh Bot commented May 15, 2026

Copy link
Copy Markdown
Contributor

LLVM Coverage Report

Metric Baseline Current Δ
Lines 84.10% 84.20% +0.10%
Functions 90.90% 91.00% +0.10%
Branches 76.60% 76.60% +0.00%

Changed lines: 100.00% (6/6) · Uncovered code

Full report · Diff report

@thevar1able thevar1able added this pull request to the merge queue May 18, 2026
Merged via the queue into master with commit af1f45f May 18, 2026
324 of 330 checks passed
@thevar1able thevar1able deleted the fix-disabled-aws-logger branch May 18, 2026 11:47
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default 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.

3 participants