Skip to content

Throws an error if iceberg_metadata_log is not configured#87250

Merged
divanik merged 2 commits intomasterfrom
divanik/remove_disks_app_interactive_from_parallel_execution
Sep 18, 2025
Merged

Throws an error if iceberg_metadata_log is not configured#87250
divanik merged 2 commits intomasterfrom
divanik/remove_disks_app_interactive_from_parallel_execution

Conversation

@divanik
Copy link
Copy Markdown
Member

@divanik divanik commented Sep 17, 2025

Closes #86681

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):

Throws an error if iceberg_metadata_log is not configured, but user tries to get debug iceberg metadata info. Fixes nullptr access

@divanik divanik added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Sep 17, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 17, 2025

Workflow [PR], commit [9753aa5]

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Sep 17, 2025
@divanik
Copy link
Copy Markdown
Member Author

divanik commented Sep 18, 2025

Flaky checks:
AST fuzzer (amd_msan) - #84186

@divanik divanik added this pull request to the merge queue Sep 18, 2025
Merged via the queue into master with commit b46bdc9 Sep 18, 2025
121 of 123 checks passed
@divanik divanik deleted the divanik/remove_disks_app_interactive_from_parallel_execution branch September 18, 2025 13:54
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 18, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Sep 18, 2025
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care! 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.

Trivial SEGV with empty Iceberg table

5 participants