Skip to content

Fix DeltaLakeMetadataDeltaKernel::prepareReadingFromFormat#86477

Merged
kssenii merged 18 commits intomasterfrom
delta-lake-fix-reading-subcolumns-2
Sep 9, 2025
Merged

Fix DeltaLakeMetadataDeltaKernel::prepareReadingFromFormat#86477
kssenii merged 18 commits intomasterfrom
delta-lake-fix-reading-subcolumns-2

Conversation

@kssenii
Copy link
Copy Markdown
Member

@kssenii kssenii commented Sep 1, 2025 •
edited
Loading

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 reading subcolumns in DeltaLake table engine. Closes #86204.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 1, 2025 •
edited
Loading

Workflow [PR], commit [bf12617]

Summary: ❌

job_name test_name status info comment
Stateless tests (arm_binary, parallel) failure
00601_kill_running_query FAIL
Integration tests (arm_binary, distributed plan, 4/4) failure
test_storage_s3_queue/test_4.py::test_list_and_delete_race FAIL

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Sep 1, 2025
PedroTadim
PedroTadim reviewed Sep 1, 2025
View reviewed changes
@kssenii kssenii force-pushed the delta-lake-fix-reading-subcolumns-2 branch from 27e1c13 to 9ab10e3 Compare September 1, 2025 16:21
@kssenii kssenii changed the title Delta lake fix reading subcolumns Fix DeltaLakeMetadataDeltaKernel::prepareReadingFromFormat Sep 5, 2025
@kssenii kssenii force-pushed the delta-lake-fix-reading-subcolumns-2 branch from eda9e1b to 26618d8 Compare September 5, 2025 14:06
@kssenii kssenii force-pushed the delta-lake-fix-reading-subcolumns-2 branch from dde7d77 to cee1496 Compare September 8, 2025 11:34
@kssenii kssenii force-pushed the delta-lake-fix-reading-subcolumns-2 branch from e1f846c to bf12617 Compare September 8, 2025 16:25
@Avogar Avogar self-assigned this Sep 8, 2025
@kssenii kssenii added this pull request to the merge queue Sep 9, 2025
Merged via the queue into master with commit 859229f Sep 9, 2025
119 of 122 checks passed
@kssenii kssenii deleted the delta-lake-fix-reading-subcolumns-2 branch September 9, 2025 15:40
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 9, 2025
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 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 9, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 mentioned this pull request Sep 9, 2025
Closed
@robot-ch-test-poll robot-ch-test-poll 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

Reviewers

@PedroTadim PedroTadim PedroTadim left review comments

@Avogar Avogar Avogar approved these changes

Assignees

@Avogar Avogar

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-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo v25.8-must-backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Logical error: 'Not found column c1.1.size0 in physical names map. There are only columns: c1' with Delta Table

6 participants

@kssenii @PedroTadim @Avogar @robot-ch-test-poll @robot-clickhouse-ci-1 @robot-clickhouse-ci-2