-
Notifications
You must be signed in to change notification settings - Fork 4.8k
HIVE-26829: Upgrade avro to 1.11.0 #3878
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
Kudos, SonarCloud Quality Gate passed! |
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.
+1 (non-binding)
I applied the patch locally. I confirmed the new Avro version in mvn dependency:tree
. I ran a full build with -Pdist -Piceberg
and confirmed that the new version was getting bundled in the distro.
Thank you for the patch, @Aggarwal-Raghav .
How I tested this: Replaced avro jars with 1.11.0: Restarted the services and was able to read and insert data into the previously created avro table. |
@Aggarwal-Raghav thanks for reporting and working on this. @cnauroth / @Aggarwal-Raghav , I have one query.? |
Hello @brahmareddybattula . I reviewed Avro release notes between 1.9.2 and 1.11.0. I didn't find any mention of backward-incompatible changes, though I don't know if that's a definitive answer. Hadoop noticed a potential incompatibility in the upgrade from 1.8.2. to 1.9.0, documented in HADOOP-13386. I'm wondering if the testing mentioned by @Aggarwal-Raghav is sufficient to show the upgrade is safe? While researching, I noticed that Avro has released 1.11.1, so if we can confirm the upgrade is safe, then perhaps we want to jump to that. I wasn't clear on what we should check with Hadoop, but if you want to start a conversation on the Hadoop dev list or tag other Hadoop contributors here, please do so. |
…axena, Stamatis Zampetakis, Chris Nauroth) Co-authored-by: Raghav Aggarwal <raghavaggarwal03.ra@gmail.com> Closes apache#3878 Closes apache#4012
…axena, Stamatis Zampetakis, Chris Nauroth) Co-authored-by: Raghav Aggarwal <raghavaggarwal03.ra@gmail.com> Closes apache#3878 Closes apache#4012
…axena, Stamatis Zampetakis, Chris Nauroth) Co-authored-by: Raghav Aggarwal <raghavaggarwal03.ra@gmail.com> Closes apache#3878 Closes apache#4012
…axena, Stamatis Zampetakis, Chris Nauroth) Co-authored-by: Raghav Aggarwal <raghavaggarwal03.ra@gmail.com> Closes apache#3878 Closes apache#4012
What changes were proposed in this pull request?
To upgrade avro version to 1.11.0
Why are the changes needed?
To address the CVE-2021-43045
Does this PR introduce any user-facing change?
No
How was this patch tested?
On local machine