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
PARQUET-1183: Add Avro builders using InputFile and OutputFile. #460
PARQUET-1183: Add Avro builders using InputFile and OutputFile. #460
Conversation
@jbonofre, could you look at this and make sure it meets your needs? |
I am struggling with git, apologies for the extra work. |
@zivanfi, could you review this? It was requested for the next release. @wernerdaehn, I'll close the other issue when this is merged. Does this meet your needs? |
Yes, certainly. And my apologies again. I promise I will get better. |
No problem, thanks for contributing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Side note: Builder(Path path) is deprecated in the superclasses ParquetReader.Builder and ParquetWriter.Builder, but it is not deprecated in the derived classes AvroParquetReader.Builder and AvroParquetWriter.Builder. Is that intentional?
Good catch, I'll mark them deprecated in the Avro API as well. |
This is based on #446. I'm adding this PR because we want to get this into 1.10.0 and haven't heard back from that PR's author.