-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-17721. Replace Guava Sets usage by Hadoop's own Sets in hadoop-yarn-project #3033
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
|
💔 -1 overall
This message was automatically generated. |
78b9066 to
f631302
Compare
|
🎊 +1 overall
This message was automatically generated. |
f631302 to
727e8c5
Compare
|
💔 -1 overall
This message was automatically generated. |
8ccb2d3 to
2bbc6c1
Compare
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
Build failure was introduced because a commit landed on hadoop-yarn used guava Sets just after this PR was raised. |
tasanuma
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. +1.
|
@virajjasani Thanks for your effort on this! |
|
I confirmed that "mvn clean install -DskipTest" passed with the latest trunk. So under hadoop-yarn-project is clean from Guava Sets now. |
…-yarn-project (apache#3033) Signed-off-by: Takanobu Asanuma <[email protected]>
No description provided.