Skip to content

Conversation

@etseidl
Copy link
Contributor

@etseidl etseidl commented Nov 4, 2025

Which issue does this PR close?

Rationale for this change

See issue

What changes are included in this PR?

Modifies a few conversion functions to account for unknown logical types

Are these changes tested?

Yes, tests are added

Are there any user-facing changes?

No

@github-actions github-actions bot added the parquet Changes to the parquet crate label Nov 4, 2025
@etseidl
Copy link
Contributor Author

etseidl commented Nov 5, 2025

Thanks @mbrobbel!

@etseidl etseidl merged commit eaca232 into apache:main Nov 5, 2025
16 checks passed
@etseidl etseidl deleted the unknown_logical_type branch November 5, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parquet reader cannot handle files with unknown logical types

2 participants