Skip to content

Do not put hive partition columns in format header - fix 87515#87528

Merged
al13n321 merged 2 commits intoClickHouse:masterfrom
arthurpassos:do_not_put_hive_columns_in_format_header
Sep 24, 2025
Merged

Do not put hive partition columns in format header - fix 87515#87528
al13n321 merged 2 commits intoClickHouse:masterfrom
arthurpassos:do_not_put_hive_columns_in_format_header

Conversation

@arthurpassos
Copy link
Contributor

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

Never put hive partition columns in the format header. Fixes #87515

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@arthurpassos
Copy link
Contributor Author

@kssenii Can you have a look?

@kssenii kssenii self-assigned this Sep 24, 2025
@kssenii kssenii added the can be tested Allows running workflows for external contributors label Sep 24, 2025
@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Sep 24, 2025

Workflow [PR], commit [6ca505e]

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Sep 24, 2025
@arthurpassos
Copy link
Contributor Author

CICD failures #86145

@al13n321 al13n321 added this pull request to the merge queue Sep 24, 2025
Merged via the queue into ClickHouse:master with commit c64eda6 Sep 24, 2025
116 of 123 checks passed
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 24, 2025
@zoomxi
Copy link
Contributor

zoomxi commented Sep 25, 2025

It seems the function test files are in the wrong location.

@arthurpassos
Copy link
Contributor Author

It seems the function test files are in the wrong location.

Ouch, that's true. It's probably because I tested this code in a non related branch and then just moved the files to the proper branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

can be tested Allows running workflows for external contributors 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.

Can not read hive partitioned parquet files with s3 and RawBlob, One formats

6 participants