Skip to content

fix segfault in DeltaLakeMetadata::getFieldValue#94627

Merged
mstetsyuk merged 2 commits intomasterfrom
fix-segfault-in-DeltaLakeMetadata-getFieldValue
Jan 22, 2026
Merged

fix segfault in DeltaLakeMetadata::getFieldValue#94627
mstetsyuk merged 2 commits intomasterfrom
fix-segfault-in-DeltaLakeMetadata-getFieldValue

Conversation

@mstetsyuk
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Critical Bug Fix (crash, data loss, RBAC) or LOGICAL_ERROR

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

Fixes https://github.com/ClickHouse/clickhouse-core-incidents/issues/1250. Likely due to a mix-up, data_type (which may be DataTypeNullable*) is static_cast-ed to DataTypeDateTime64*. Instead, we should cast check_type (the nested type variable).

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jan 19, 2026

Workflow [PR], commit [f0476d0]

Summary:

job_name test_name status info comment
Stateless tests (amd_tsan, s3 storage, parallel) failure
03562_parallel_replicas_remote_with_cluster FAIL cidb, issue
Integration tests (arm_binary, distributed plan, 1/4) failure
test_max_temporary_data_size_on_disk/test.py::test FAIL cidb, issue
BuzzHouse (amd_debug) failure
Logical error: 'Inconsistent AST formatting: the query: (STID: 1941-1bfa) FAIL cidb, issue

@clickhouse-gh clickhouse-gh bot added pr-critical-bugfix pr-must-backport Pull request should be backported intentionally. Use this label with great care! labels Jan 19, 2026
@SmitaRKulkarni SmitaRKulkarni self-assigned this Jan 20, 2026
@mstetsyuk mstetsyuk added this pull request to the merge queue Jan 22, 2026
Merged via the queue into master with commit a1f135b Jan 22, 2026
128 of 132 checks passed
@mstetsyuk mstetsyuk deleted the fix-segfault-in-DeltaLakeMetadata-getFieldValue branch January 22, 2026 13:38
@robot-ch-test-poll3 robot-ch-test-poll3 added pr-synced-to-cloud The PR is synced to the cloud repo pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Jan 22, 2026
robot-clickhouse added a commit that referenced this pull request Jan 22, 2026
Cherry pick #94627 to 25.8: fix segfault in DeltaLakeMetadata::getFieldValue
robot-clickhouse added a commit that referenced this pull request Jan 22, 2026
Cherry pick #94627 to 25.10: fix segfault in DeltaLakeMetadata::getFieldValue
robot-clickhouse added a commit that referenced this pull request Jan 22, 2026
Cherry pick #94627 to 25.11: fix segfault in DeltaLakeMetadata::getFieldValue
robot-clickhouse added a commit that referenced this pull request Jan 22, 2026
Cherry pick #94627 to 25.12: fix segfault in DeltaLakeMetadata::getFieldValue
clickhouse-gh bot added a commit that referenced this pull request Jan 22, 2026
Backport #94627 to 25.11: fix segfault in DeltaLakeMetadata::getFieldValue
clickhouse-gh bot added a commit that referenced this pull request Jan 22, 2026
Backport #94627 to 25.8: fix segfault in DeltaLakeMetadata::getFieldValue
mstetsyuk added a commit that referenced this pull request Jan 23, 2026
Backport #94627 to 25.12: fix segfault in DeltaLakeMetadata::getFieldValue
mstetsyuk added a commit that referenced this pull request Jan 23, 2026
Backport #94627 to 25.10: fix segfault in DeltaLakeMetadata::getFieldValue
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jan 23, 2026
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-critical-bugfix 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.

5 participants