-
Notifications
You must be signed in to change notification settings - Fork 9.2k
YARN-11257. Add junit5 dependency to hadoop-yarn-server-timeline-pluginstorage to fix few unit test failure #4770
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
…instorage to some unit test failure
aajisaka
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 pending Jenkins. Thanks.
|
💔 -1 overall
This message was automatically generated. |
aajisaka
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.
Cancelling my +1. The tests were not actually running. Could you update the tests to use junit 5? Sorry for back and forth.
|
You've already created #4771 and I'll look into this. |
Thanks |
I cant access this link. Can you share what's the error. In my local - even with this change, tests were passing. |
|
Closing this PR as we have this follow-up PR #4771 |
|
Sorry I pasted the wrong link. Can you access to https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4770/1/artifact/out/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timeline-pluginstorage.txt ? |
|
Yes. Now I can access. Thanks for sharing. |
Description of PR
Add junit5 dependency to hadoop-yarn-server-timeline-pluginstorage to some unit test failure
JIRA - YARN-11257
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?