Skip to content

Sync parquet.thrift with apache/parquet-format master (ALP, FILE) - #87

Merged
alexey-milovidov merged 1 commit into
ClickHouse/apache-arrow-25.0.0from
ClickHouse/parquet-format-alp-file
Oct 11, 2026
Merged

alexey-milovidov merged 1 commit into
ClickHouse/apache-arrow-25.0.0from
ClickHouse/parquet-format-alp-file

Conversation

@alexey-milovidov

Copy link
Copy Markdown
Member

Syncs cpp/src/parquet/parquet.thrift with apache/parquet-format master. This adds the ALP encoding, the FILE logical type and INT96_TIMESTAMP_ORDER. The code in cpp/src/generated is regenerated with Thrift 0.25.0, the runtime version ClickHouse uses in contrib/thrift.

ClickHouse needs this so its native Parquet reader and writer can support ALP and FILE.

🤖 Generated with Claude Code

Adds the `ALP` encoding, the `FILE` logical type (`FileType`) and the
`INT96_TIMESTAMP_ORDER` column order. The generated code is regenerated with
the Thrift compiler 0.25.0 (matching the Thrift runtime used by ClickHouse) via
`thrift --gen cpp:moveable_types,templates -out src/generated src/parquet/parquet.thrift`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Thanks for opening a pull request!

This pull request has been automatically converted to a draft because its title doesn't match Arrow's required format.

If this is not a minor PR, could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

After updating the title, you can mark the pull request as ready for review.

See also:

@alexey-milovidov alexey-milovidov self-assigned this Oct 11, 2026
@alexey-milovidov
alexey-milovidov merged commit 363af92 into ClickHouse/apache-arrow-25.0.0 Oct 11, 2026
48 of 101 checks passed
Ergus pushed a commit to Ergus/ClickHouse that referenced this pull request Oct 14, 2026
…ormat master

This brings the `ALP` encoding and the `FILE` logical type into the generated
Thrift code used by the native Parquet reader and writer.

Related: ClickHouse/arrow#87

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Ergus pushed a commit to Ergus/ClickHouse that referenced this pull request Oct 14, 2026
The tree is identical; this points the submodule at the commit on the
`ClickHouse/apache-arrow-25.0.0` branch.

Related: ClickHouse/arrow#87

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant