-
Notifications
You must be signed in to change notification settings - Fork 1k
Open
Labels
enhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelogparquetChanges to the parquet crateChanges to the parquet crate
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The parquet::format
module contains auto generated code for parsing data in the parquet.thrift format
After we merge #8530 from @etseidl these structures will no longer be needed and their presence is ust confusing
I tested it out a bit locally and the only place the thrift structures are now used is in the parquet-format
tool and some benchmarks

Describe the solution you'd like
Delete the parquet::format
module (parquet/src/format.rs
), and update the tools to use the new thrift structures
Describe alternatives you've considered
Additional context
Metadata
Metadata
Assignees
Labels
enhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelogparquetChanges to the parquet crateChanges to the parquet crate