-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Build: Update Parquet and Avro dependencies #5188
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
Conversation
| @@ -1,12 +1,12 @@ | |||
| org.slf4j:* = 1.7.25 | |||
| org.apache.avro:avro = 1.10.1 | |||
| org.apache.avro:avro = 1.10.2 | |||
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.
would it make sense to update to avro 1.11.0 since it has been around for a while already?
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.
I think the purpose of updating Avro was to keep parquet-avro and avro in sync. That seems like a reasonable choice to me.
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.
Makes sense, +1
kbendick
left a comment
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.
Thanks for getting this working @rdblue!
|
For reference, the changelog from 1.12.2 to 1.12.3 is here: apache/parquet-java@apache-parquet-1.12.2...apache-parquet-1.12.3 |
This is based on #4951 from Kyle, but attempts to fix the current test failures.