Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

YARN-11793: Replace Grizzly with Jetty in Jersey Test Framework#7497

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
brumi1024 merged 6 commits intoapache:trunkfromsusheelgupta7:YARN-11793-1
May 12, 2025

Conversation

susheelgupta7
Copy link
Contributor

@susheelgupta7susheelgupta7 commentedMar 11, 2025•
edited
Loading

Description of PR

JIRA:YARN-11793: Replace Grizzly with Jetty in Jersey Test Framework
Replaced jersey-test-framework-provider-grizzly2 with jersey-test-framework-provider-jetty to better align with Hadoop's production and provide more real HTTP testing environment.

jersey-test-framework-provider-inmemory was avoided as it doesn't properly simulate real HTTP behavior and it is not consistent with the production setup. Since RMWebServices depends on actual HTTP interactions, using Jetty makes the testing framework more reliable and closer to real-world scenarios.

How was this patch tested?

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion underASF 2.0?
  • If applicable, have you updated theLICENSE,LICENSE-binary,NOTICE-binary files?

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 19sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep6m 9sMaven dependency ordering for branch
+1 💚mvninstall18m 43strunk passed
+1 💚compile8m 24strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 26strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite13m 34strunk passed
+1 💚javadoc5m 20strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 56strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient29m 28sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 23sMaven dependency ordering for patch
+1 💚mvninstall31m 30sthe patch passed
+1 💚compile8m 0sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac8m 0sthe patch passed
+1 💚compile7m 20sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac7m 20sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite8m 44sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 16sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 57sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
-1 ❌shadedclient29m 49spatch has errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit497m 21s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 50sThe patch does not generate ASF License warnings.
670m 4s
ReasonTests
Failed junit testshadoop.yarn.client.api.impl.TestYarnClient
hadoop.yarn.client.api.impl.TestAMRMClientPlacementConstraints
hadoop.yarn.client.api.impl.TestAMRMClient
hadoop.yarn.client.api.impl.TestAMRMProxy
hadoop.yarn.client.api.impl.TestNMClient
hadoop.yarn.client.api.impl.TestOpportunisticContainerAllocationE2E
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/1/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux 556bd035e47c 5.15.0-130-generic#140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /e40e668
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/1/testReport/
Max. process+thread count3997 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/1/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

@susheelgupta7
Copy link
ContributorAuthor

@cnauroth Could you please review this patch.

@slfan1989
Copy link
Contributor

@susheelgupta7 I think this upgrade is reasonable for me, but it may need to wait until the Yarn JUnit5 upgrade is completed and merged. My expected timeline is about one week.

susheel-gupta reacted with thumbs up emoji

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec6m 49sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 1scodespell was not available.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+0 🆗xmllint0m 1sxmllint was not available.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep5m 53sMaven dependency ordering for branch
+1 💚mvninstall18m 38strunk passed
+1 💚compile8m 26strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 13strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite13m 12strunk passed
+1 💚javadoc5m 23strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 58strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient29m 57sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 23sMaven dependency ordering for patch
+1 💚mvninstall31m 12sthe patch passed
+1 💚compile8m 8sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac8m 8sthe patch passed
+1 💚compile7m 9sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac7m 9sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite12m 5sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 17sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 46sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
-1 ❌shadedclient30m 48spatch has errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit492m 1s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 56sThe patch does not generate ASF License warnings.
674m 43s
ReasonTests
Failed junit testshadoop.yarn.client.api.impl.TestYarnClient
hadoop.yarn.client.api.impl.TestAMRMClientPlacementConstraints
hadoop.yarn.client.api.impl.TestAMRMClient
hadoop.yarn.client.api.impl.TestAMRMProxy
hadoop.yarn.client.api.impl.TestNMClient
hadoop.yarn.client.api.impl.TestOpportunisticContainerAllocationE2E
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/2/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux f5d78f218299 5.15.0-130-generic#140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /d6ad25e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/2/testReport/
Max. process+thread count3913 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/2/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 20sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 1scodespell was not available.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+0 🆗xmllint0m 1sxmllint was not available.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep6m 12sMaven dependency ordering for branch
+1 💚mvninstall18m 27strunk passed
+1 💚compile8m 28strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 15strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite13m 42strunk passed
+1 💚javadoc5m 18strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 55strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient29m 35sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 23sMaven dependency ordering for patch
+1 💚mvninstall31m 11sthe patch passed
+1 💚compile8m 4sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac8m 4sthe patch passed
+1 💚compile7m 17sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac7m 17sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite11m 32sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 16sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 56sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
-1 ❌shadedclient29m 47spatch has errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit492m 23s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 54sThe patch does not generate ASF License warnings.
667m 21s
ReasonTests
Failed junit testshadoop.yarn.client.api.impl.TestYarnClient
hadoop.yarn.client.api.impl.TestAMRMClientPlacementConstraints
hadoop.yarn.client.api.impl.TestAMRMClient
hadoop.yarn.client.api.impl.TestAMRMProxy
hadoop.yarn.client.api.impl.TestNMClient
hadoop.yarn.client.api.impl.TestOpportunisticContainerAllocationE2E
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/3/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux 058364d21e35 5.15.0-130-generic#140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /d86a046
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/3/testReport/
Max. process+thread count3930 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/3/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 20sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 1scodespell was not available.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+0 🆗xmllint0m 1sxmllint was not available.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep5m 37sMaven dependency ordering for branch
+1 💚mvninstall20m 10strunk passed
+1 💚compile8m 26strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 28strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite10m 31strunk passed
+1 💚javadoc5m 25strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 55strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient30m 11sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 24sMaven dependency ordering for patch
+1 💚mvninstall31m 5sthe patch passed
+1 💚compile7m 58sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac7m 58sthe patch passed
+1 💚compile8m 26sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac8m 26sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite12m 11sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 27sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 57sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient30m 21spatch has no errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit493m 30s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 48sThe patch does not generate ASF License warnings.
669m 15s
ReasonTests
Failed junit testshadoop.yarn.client.api.impl.TestYarnClient
hadoop.yarn.client.api.impl.TestAMRMClientPlacementConstraints
hadoop.yarn.client.api.impl.TestAMRMClient
hadoop.yarn.client.api.impl.TestAMRMProxy
hadoop.yarn.client.api.impl.TestNMClient
hadoop.yarn.client.api.impl.TestOpportunisticContainerAllocationE2E
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/4/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux 7d9f51afed99 5.15.0-130-generic#140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /10a4903
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/4/testReport/
Max. process+thread count3546 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/4/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@cnaurothcnauroth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@susheelgupta7 , thank you for the new patch. I agree with@slfan1989 on sequencing this after the YARN JUnit 5 upgrades. Then, we can check for a clean test run here.

susheelgupta7 reacted with thumbs up emoji
@slfan1989
Copy link
Contributor

@susheelgupta7 , thank you for the new patch. I agree with@slfan1989 on sequencing this after the YARN JUnit 5 upgrades. Then, we can check for a clean test run here.

@cnauroth Thank you for your message! Starting today, I will continue with the YARN JUnit 5 upgrade. I was a bit tied up with work over the past few days, which caused a slight delay.

cc:@susheelgupta7

susheelgupta7 reacted with thumbs up emoji

@susheelgupta7
Copy link
ContributorAuthor

@slfan1989 May I ask if we could prioritize merging the upgrade from JUnit 4 to JUnit 5 a bit sooner. The reason is that grizzly-http is pulling inCVE-2017-1000028. It is necessary to address this and mitigate the risk. Please let me know your thoughts—really appreciate your help!.

@slfan1989
Copy link
Contributor

@slfan1989 May I ask if we could prioritize merging the upgrade from JUnit 4 to JUnit 5 a bit sooner. The reason is that grizzly-http is pulling inCVE-2017-1000028. It is necessary to address this and mitigate the risk. Please let me know your thoughts—really appreciate your help!.

@susheelgupta7 Thank you for your continued attention to this issue! Currently, only the yarn-router module and some code related to the hadoop-common module remain to be upgraded. I will make every effort to complete this upgrade within this week.

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 19sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep5m 36sMaven dependency ordering for branch
+1 💚mvninstall19m 47strunk passed
+1 💚compile9m 14strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 47strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite13m 31strunk passed
+1 💚javadoc5m 23strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 57strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient29m 32sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 25sMaven dependency ordering for patch
+1 💚mvninstall31m 27sthe patch passed
+1 💚compile8m 0sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac8m 0sthe patch passed
+1 💚compile7m 16sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac7m 16sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite11m 37sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 22sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc5m 0sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient30m 30spatch has no errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit525m 30s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 51sThe patch does not generate ASF License warnings.
703m 27s
ReasonTests
Failed junit testshadoop.yarn.client.api.impl.TestYarnClient
hadoop.yarn.client.api.impl.TestAMRMClientPlacementConstraints
hadoop.yarn.client.api.impl.TestAMRMClient
hadoop.yarn.client.api.impl.TestAMRMProxy
hadoop.yarn.client.api.impl.TestNMClient
hadoop.yarn.client.api.impl.TestOpportunisticContainerAllocationE2E
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/5/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux 9bd77a24d3af 5.15.0-136-generic#147-Ubuntu SMP Sat Mar 15 15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /4df420e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/5/testReport/
Max. process+thread count3484 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/5/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

@susheelgupta7
Copy link
ContributorAuthor

Currently, only the yarn-router module and some code related to the hadoop-common module remain to be upgraded

@slfan1989 Could you please review this patch please, if the upgrade of above modules are completed ??.
The test failures are unrelated. These tests are passing with or without the patch locally.

@slfan1989
Copy link
Contributor

@susheelgupta7 Thank you for your contribution! We have fixed the unit test errors. Please trigger the compilation again.

@hadoop-yetus
Copy link

💔-1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 20sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any@author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗mvndep8m 11sMaven dependency ordering for branch
+1 💚mvninstall18m 40strunk passed
+1 💚compile8m 21strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚compile7m 26strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚mvnsite14m 23strunk passed
+1 💚javadoc5m 33strunk passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc4m 59strunk passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient30m 0sbranch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 24sMaven dependency ordering for patch
+1 💚mvninstall31m 48sthe patch passed
+1 💚compile8m 6sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javac8m 6sthe patch passed
+1 💚compile7m 29sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚javac7m 29sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚mvnsite11m 21sthe patch passed
+1 💚shellcheck0m 0sNo new issues.
+1 💚javadoc5m 25sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04
+1 💚javadoc5m 2sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
+1 💚shadedclient31m 14spatch has no errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌unit560m 28s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 55sThe patch does not generate ASF License warnings.
740m 57s
ReasonTests
Failed junit testshadoop.hdfs.server.federation.router.TestRouterMountTableCacheRefresh
hadoop.mapred.gridmix.TestLoadJob
hadoop.mapred.gridmix.TestSleepJob
hadoop.mapred.gridmix.TestGridmixSubmission
SubsystemReport/Notes
DockerClientAPI=1.49 ServerAPI=1.49 base:https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/6/artifact/out/Dockerfile
GITHUB PR#7497
Optional Testsdupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs
unameLinux e9acd91e1a3b 5.15.0-136-generic#147-Ubuntu SMP Sat Mar 15 15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/bin/hadoop.sh
git revisiontrunk /cfd3233
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu120.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/6/testReport/
Max. process+thread count3586 (vs. ulimit of 5500)
modulesC: hadoop-project hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-globalpolicygenerator hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-resourceestimator hadoop-client-modules/hadoop-client-runtime hadoop-client-modules/hadoop-client-minicluster . U: .
Console outputhttps://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7497/6/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered byApache Yetus 0.14.0https://yetus.apache.org

This message was automatically generated.

@slfan1989
Copy link
Contributor

@susheelgupta7 Thanks for the contribution! The unit test error that occurred is unrelated to the changes in this PR.

@brumi1024
Copy link
Member

Thanks@susheelgupta7 for the patch,@slfan1989 for the review, merging to trunk.

@brumi1024brumi1024 merged commit47ad1c0 intoapache:trunkMay 12, 2025
1 of 4 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@cnaurothcnaurothcnauroth left review comments

@slfan1989slfan1989slfan1989 approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@susheelgupta7@hadoop-yetus@slfan1989@brumi1024@cnauroth
[8]ページ先頭
©2009-2025 Movatter.jp