parquet-java
parquet-java copied to clipboard
Add more constructors to ParquetFileReader
Describe the enhancement requested
It seems that we are moving towards InputFile instead of HadoopConf and path (indicated by the deprecation notice), but the constructors using InputFile are missing important ones, such as public ParquetFileReader(InputFile file, ParquetMetadata footer,ParquetReadOptions options, SeekableInputStream f) where an external footer can be passed in.
If it seems ok, I would like to add some additional constructors to bring the InputFile-based ones to parity with HadoopConf-based ones.
Component(s)
No response
I think it makes sense to add this. Please feel free to create a PR.
Can I suggest the builder pattern, allows for more evolution over time and allows every option to have a default value
@steveloughran I have the same feeling. For the reader we already have ParquetReadOptions. The writer does not even have this parity.