-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HADOOP-18342: upgrade to avro 1.11.0 #4581
Conversation
💔 -1 overall
This message was automatically generated. |
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.
Avro upgrade creates some problem. It got a -1 from @xkrogen here:
#4580 (comment)
Unless there is any CVE or so, I think we should stay away from it
I'll close this then |
Just to be clear, I am not generally -1 on Avro upgrades. #4580 would have bumped the Avro major version in a maintenance release of Hadoop; a major Avro version bump is way too big of a change to go into a maintenance release. I would not take the same stack for an upgrade in trunk, which would go into a minor release. Though notably Avro 1.11.0 has at least one major performance issue, which we have found to be severe enough internally at LinkedIn to specifically disallow use of this version: AVRO-3243. So I would recommend that we avoid any Avro 1.11 adoption until 1.11.1 is available. |
Description of PR
https://issues.apache.org/jira/browse/HADOOP-18342
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?