Skip to content

Use relevant column mapper when reading an iceberg table #85829

Merged
divanik merged 6 commits intomasterfrom
divanik/fix_wrong_column_mapper
Aug 25, 2025
Merged

Use relevant column mapper when reading an iceberg table #85829
divanik merged 6 commits intomasterfrom
divanik/fix_wrong_column_mapper

Conversation

@divanik
Copy link
Member

@divanik divanik commented Aug 18, 2025

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

Mapping between iceberg source ids and parquet names was not adjusted to the schema when the parquet file was written. This PR processes schema relevant for each iceberg data file, not a current one

@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Aug 18, 2025

Workflow [PR], commit [0c8faef]

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Aug 18, 2025
@kssenii kssenii self-assigned this Aug 19, 2025
@divanik
Copy link
Member Author

divanik commented Aug 25, 2025

@divanik divanik added this pull request to the merge queue Aug 25, 2025
Merged via the queue into master with commit 954a0d7 Aug 25, 2025
238 of 240 checks passed
@divanik divanik deleted the divanik/fix_wrong_column_mapper branch August 25, 2025 17:48
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 25, 2025
@divanik divanik added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Sep 4, 2025
robot-clickhouse-ci-2 added a commit that referenced this pull request Sep 4, 2025
Cherry pick #85829 to 25.8: Use relevant column mapper when reading an iceberg table
robot-clickhouse added a commit that referenced this pull request Sep 4, 2025
@robot-ch-test-poll robot-ch-test-poll 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 4, 2025
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Sep 4, 2025
clickhouse-gh bot added a commit that referenced this pull request Sep 4, 2025
Backport #85829 to 25.8: Use relevant column mapper when reading an iceberg table
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.

6 participants