Skip to content

Revert "Merge pull request #89750 - parquet_footer_cache"#96795

Merged
alexey-milovidov merged 1 commit intomasterfrom
revert-parquet-footer-cache
Feb 13, 2026
Merged

Revert "Merge pull request #89750 - parquet_footer_cache"#96795
alexey-milovidov merged 1 commit intomasterfrom
revert-parquet-footer-cache

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented Feb 13, 2026

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Revert #89750 (parquet footer cache) which causes a LOGICAL_ERROR exception when reading Parquet from S3 with the default settings (use_native_reader_v3 = false). The metadata-aware format creator only handled the use_native_reader_v3 path and threw a LOGICAL_ERROR for the default Arrow reader path, triggered whenever S3 storage provides blob metadata.

https://s3.amazonaws.com/clickhouse-test-reports/json.html?REF=master&sha=ecd86eea3e7db650c38e4144690d7d877f7e9f6b&name_0=MasterCI&name_1=Stress%20test%20%28arm_asan%2C%20s3%29&name_1=Stress%20test%20%28arm_asan%2C%20s3%29

…footer_cache"

This reverts commit 5243ef7, reversing
changes made to 6e3a5ad.
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Feb 13, 2026

Workflow [PR], commit [223e53c]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 13, 2026
@alexey-milovidov alexey-milovidov self-assigned this Feb 13, 2026
@alexey-milovidov alexey-milovidov merged commit e91859b into master Feb 13, 2026
0 of 2 checks passed
@alexey-milovidov alexey-milovidov deleted the revert-parquet-footer-cache branch February 13, 2026 00:04
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog 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.

2 participants