Skip to content

Fix 'Invalid dict indices bit width: 0' in parquet reader v3#87258

Merged
al13n321 merged 2 commits intomasterfrom
pq0
Sep 19, 2025
Merged

Fix 'Invalid dict indices bit width: 0' in parquet reader v3#87258
al13n321 merged 2 commits intomasterfrom
pq0

Conversation

@al13n321
Copy link
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Turns out 0 is a perfectly valid bit width.

(No test because I don't have an easy way to generate a test file. The one I looked at was 500 MB, produced by parquet-mr version 1.15.2 (build 859eac165b08f927fa14590c33bc5f476405fb68).)

@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Sep 17, 2025

Workflow [PR], commit [6fdb15a]

Summary:

job_name test_name status info comment
Integration tests (arm_binary, distributed plan, 3/4) failure
test_scheduler_cpu_preemptive/test.py::test_independent_pools FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 17, 2025
@rienath rienath self-assigned this Sep 18, 2025
Copy link
Member

@rienath rienath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Looks like CI is red due to flaky tests. Maybe worth it to merge master to make sure that this is indeed the case

@al13n321 al13n321 added this pull request to the merge queue Sep 19, 2025
Merged via the queue into master with commit ba64fe4 Sep 19, 2025
121 of 123 checks passed
@al13n321 al13n321 deleted the pq0 branch September 19, 2025 19:19
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 19, 2025
mkmkme pushed a commit to Altinity/ClickHouse that referenced this pull request Feb 9, 2026
Fix 'Invalid dict indices bit width: 0' in parquet reader v3
zvonand added a commit to Altinity/ClickHouse that referenced this pull request Feb 15, 2026
25.8.16 Backport of ClickHouse#87258 - Fix 'Invalid dict indices bit width: 0' in parquet reader v3
mkmkme pushed a commit to Altinity/ClickHouse that referenced this pull request Feb 16, 2026
Fix 'Invalid dict indices bit width: 0' in parquet reader v3
zvonand added a commit to Altinity/ClickHouse that referenced this pull request Feb 18, 2026
Antalya 25.8 Backport of ClickHouse#87258 - Fix 'Invalid dict indices bit width: 0' in parquet reader v3
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.

4 participants