-
Notifications
You must be signed in to change notification settings - Fork 9.2k
YARN-11793: Remove grizzly-http-* dependencies from Hadoop to avoid t… #7480
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
…ransitive inclusions.
|
💔 -1 overall
This message was automatically generated. |
cnauroth
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.
+1. I ran mvn dependency:tree and saw there are still org.glassfish.grizzly dependencies. These dependencies aren't present after applying the patch. I'll commit this shortly.
|
I have committed this to trunk. Thank you, @susheelgupta7 . |
|
@cnauroth I found this pr will cause unit test errors. |
|
@slfan1989 , sorry, I can get online later and revert this. |
|
@slfan1989 , I reverted this from trunk and branch-3.4. @susheelgupta7 , please feel free to reopen YARN-11793 if you want to make another attempt at this patch. |
…ransitive inclusions. Closes apache#7480 Signed-off-by: Chris Nauroth <cnauroth@apache.org>
…ransitive inclusions.
Description of PR
How was this patch tested?
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?