-
Notifications
You must be signed in to change notification settings - Fork 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-13464. Upgrade Gson to 2.8.9 #2524
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@medb are you still working on this? There is vulnerability issue reported for gson before 2.8.9 |
@manuzhang sure, updated PR to Gson 2.8.9 |
💔 -1 overall
This message was automatically generated. |
Change-Id: Ifd3fb9ec6ebfc8874bb799bc198219511fe55a2f Update pom.xml Update pom.xml
💔 -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.
+1, thanks for the contribution. I'm going to merge this to trunk, branch-3.3 and branch-3.2.
The -1 from pre-commit was due to lack of new tests, which isn't relevant to this dependency upgrade patch.
Change-Id: Ifd3fb9ec6ebfc8874bb799bc198219511fe55a2f Update pom.xml Update pom.xml (cherry picked from commit 7950548)
Hi @medb - In the next time, would you remove the Change-Id when creating PR? Thanks. |
@aajisaka sure, will do. |
Change-Id: Ifd3fb9ec6ebfc8874bb799bc198219511fe55a2f Update pom.xml Update pom.xml (cherry picked from commit 7950548)
Change-Id: Ifd3fb9ec6ebfc8874bb799bc198219511fe55a2f Update pom.xml Update pom.xml
Upgrading the Gson library to 2.8.9 (latest version at the moment).