-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-17341. Upgrade commons-codec to 1.15 #2428
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
jojochuang
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.
Looks good. Do you have a target version in mind?
|
Thank you, @jojochuang . Since the previous one, HADOOP-17033 , has a target version |
|
💔 -1 overall
This message was automatically generated. |
|
Could you review this too, @sunchao ? |
sunchao
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.
LGTM. Thanks @dongjoon-hyun
|
Thank you, @sunchao ! |
|
Gentle ping, @sunchao and @jojochuang ~ |
|
Done. Thanks both! |
|
Thank you so much! |
Change-Id: Iab26db901570b507ab25ddbf316a9579a9e92620 Reviewed-by: Chao Sun <sunchao@apache.org> Reviewed-by: Wei-Chiu Chuang <weichiu@apache.org>
Change-Id: Iab26db901570b507ab25ddbf316a9579a9e92620 Reviewed-by: Chao Sun <sunchao@apache.org> Reviewed-by: Wei-Chiu Chuang <weichiu@apache.org>
This PR aims to upgrade
commons-codecto 1.15.