-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-19617. [JDK17] Remove JUnit4 Dependency (YARN) . #7798
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. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
The relevant unit tests enabled the The error message is as follows:
|
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
I have carefully reviewed the relevant unit tests and found that they all pass locally. I believe the issue may be related to the low version of the Surefire plugin currently in use. Steve has updated the version of the Surefire plugin, and we hope this change will resolve the current issue. |
I've reviewed the latest test report, and currently, there is one unit test that timed out. I have adjusted the timeout and am waiting for feedback from Yetus. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
The unit tests pass successfully locally, but there are issues on
hadoop-yarn-server-resourcemanager links org.apache.hadoop.yarn.server.resourcemanager.TestAppManager Encountered a thread crash issue in In local testing, TestAppManager executes successfully. ![]()
hadoop-yarn-server-router links org.apache.hadoop.yarn.server.router.subcluster.capacity.TestYarnFederationWithCapacityScheduler The Yetus test report shows a timeout, but the issue cannot be reproduced locally. Unit tests execute successfully on the local. ![]() |
💔 -1 overall
This message was automatically generated. |
I hope to merge this PR as the overall compilation and unit test results are mostly as expected. Although there are a few unit tests experiencing stack overflow issues, everything works as expected in local testing. Through this testing, we identified some unit test issues in the hadoop-yarn-applications-distributedshell module, which have been fixed. |
The most recent unit test shows that all tests have passed, but the page does not display the link. I plan to merge this PR first to facilitate the continued development of other JDK 17 upgrade-related features. |
Description of PR
JIRA: HADOOP-19617. [JDK17] Remove JUnit4 Dependency(YARN).
How was this patch tested?
Junit Test.
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?