-
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-19034. Fix Download Maven Url Not Found. #6438
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.
+1.
tested the curl command locally and it works
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. +1. Check this link and command works fine at local.
Merged. Thanks @slfan1989 and @steveloughran . |
…by Shilun Fan. Reviewed-by: Steve Loughran <stevel@apache.org> Signed-off-by: He Xiaoqiao <hexiaoqiao@apache.org> (cherry picked from commit 0f8b74b)
…by Shilun Fan. Reviewed-by: Steve Loughran <stevel@apache.org> Signed-off-by: He Xiaoqiao <hexiaoqiao@apache.org> (cherry picked from commit 0f8b74b)
@Hexiaoqiao @steveloughran Thanks for reviewing the code! |
FWIW I think I just aggregated things like this into the general "release" JIRA so been able to push stuff through without so much effort. |
…buted by Shilun Fan. Reviewed-by: Steve Loughran <stevel@apache.org> Signed-off-by: He Xiaoqiao <hexiaoqiao@apache.org>
Description of PR
JIRA: HADOOP-19034. Fix Download Maven Url Not Found.
In the process of preparing Hadoop 3.4.0 Release, I found that when opening the link to download maven, it will prompt not found. We need to fix this issue.
wget 'https://dlcdn.apache.org/maven/maven-3/3.6.3/binaries/apache-maven-3.6.3-bin.tar.gz'
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?