-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-17288. Use shaded guava from thirdparty. (branch-3.3) #2505
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
The test results are at Not sure, why there is no comment from jenkins here |
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.
Skimmed. LGTM. What did you have to change from master PR?
Failures don't look to be obviously related. Do you recognize any of these failures as flakies? Regular failings? I tried to retrigger the build but don't see how. Perhaps try a new push to see what failures we see in a re-run?
Jenkinsfile
Outdated
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.
This because so much has been touched? We need the longer time to build?
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 increased it, Luckily got the result this time
💔 -1 overall
This message was automatically generated. |
44c1519
to
c63b2e5
Compare
💔 -1 overall
This message was automatically generated. |
@saintstack Thanx for the review. There is a javadoc build failure, that is there in branch-3.3 without my patch as well for aws, |
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 me
(We have conflicts now?)
…pache#2434) Reviewed-by: Ayush Saxena <ayushsaxena@apache.org> Reviewed-by: Wei-Chiu Chuang <weichiu@apache.org>
c63b2e5
to
b84bc63
Compare
Thanx @saintstack for the review, I have rebased and merged this to branch-3.3 |
💔 -1 overall
This message was automatically generated. |
https://issues.apache.org/jira/browse/HADOOP-17288