-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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-19090. Use protobuf-java 3.23.4. #6593
Conversation
💔 -1 overall
This message was automatically generated. |
The 1 broken test seems unrelated:
|
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.
the change is merged to hadoop-thirdparty & we have a automated build which publishes the snapshot version of that, can we use that, if the build is green we can merge this to get the tests fixed
update fix groupId fix repository url
53109b7
to
9a7e4e6
Compare
Switched PR to use the Apache snapshot |
Thanks @pjfanning . LGTM. Let's wait what CI will say. |
💔 -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.
LGTM
💔 -1 overall
This message was automatically generated. |
Some of the test issues relate to
|
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 from my side.
we ready to get this into branch-3.4? |
@steveloughran this PR depends on a snapshot version of hadoop-thirdparty. Can we do a release of hadoop-thirdparty and then backport the uptake of the future hadoop-thirdparty release to the 3.4 branch? |
are we up to date with other dependencies for a new thirdparty release? we should really bump them all up and then rebuild it. |
Guava has a slightly newer release 33.0.0-jre. Is this upgrade needed though? |
It seems that |
@zhangshuyan0 it is in the Apache Snapshot repo. See the version number: 1.3.0-SNAPHOT. See https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6593/3 |
@pjfanning Thanks for your reply. The reason I couldn't compile successfully was due to my local network problem. |
…nning. (#6663) Signed-off-by: Shilun Fan <slfan1989@apache.org>
Description of PR
DRAFT - See HADOOP-19090
The hadoop-thirdparty change is not merged yet so I am temporarily testing with a snapshot that I published with my groupId.
How was this patch tested?
Local build
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?