etseidl commented on issue #8643: URL: https://github.com/apache/arrow-rs/issues/8643#issuecomment-3464086872
> > The ParquetMetaDataReader and ParquetMetaDataPushDecoder manage their own set of options. > > I looked at the code and I think the only real options now are "should I read the page index": > > * https://docs.rs/parquet/latest/src/parquet/file/metadata/reader.rs.html#66-75 > > * https://docs.rs/parquet/latest/src/parquet/file/metadata/push_decoder.rs.html#221-232 > > > What if we made a new(!!!) options object like > > struct ParquetMetadataOptions { > ... > } > > And then passed it to the `ParquetMetadataReader` (which now just wraps the `MetadataPushDecoder`)? That's what I was thinking...but then who creates it and where? I'm starting to trace back from the metadata parsers to their calling sites to see what might make the most sense. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
