-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-19594. Bump Maven 3.9.11 #7760
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
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.
Nice. Thank you.
|
💔 -1 overall
This message was automatically generated. |
|
I just found that the Maven installation is different from each OS docker image ... |
|
never used mvnw; not sure if I'd rush to, but for docker images it makes a lot of sense as a single control point -and makes it trivial for PRs to test new versions, rollback etc |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
|
(!) A patch to the testing environment has been detected. |
7c68d78 to
14e9444
Compare
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
1 similar comment
|
💔 -1 overall
This message was automatically generated. |
377e83d to
a08db49
Compare
|
💔 -1 overall
This message was automatically generated. |
a08db49 to
058a0a6
Compare
|
💔 -1 overall
This message was automatically generated. |
|
@steveloughran @dongjoon-hyun I plan to merge this PR to upgrade the Maven version. Do you have any other feedback? |
|
If there are no comments by the end of today, this PR will be merged. |
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.
I have no other comments. Thank you, @pan3793, @slfan1989 and all.
|
@pan3793 Thanks for the contribution! @dongjoon-hyun Thanks for the review! |
Description of PR
HADOOP-19594. Bump Maven 3.9.11.
Maven prior 3.8 is EOL, bump Maven from 3.6.3 to latest 3.9.11
This PR also unifies the Maven version in all Dockerfiles.
How was this patch tested?
CI.
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?