Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more constructors to ParquetFileReader #3141

Open
yuzhu opened this issue Jan 30, 2025 · 1 comment
Open

Add more constructors to ParquetFileReader #3141

yuzhu opened this issue Jan 30, 2025 · 1 comment

Comments

@yuzhu
Copy link

yuzhu commented Jan 30, 2025

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

@wgtmac
Copy link
Member

wgtmac commented Feb 3, 2025

I think it makes sense to add this. Please feel free to create a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants