Skip to content

YARN-11897: Make NMWebservices API backward-compatible after Jersey 2…#8227

Open
Hean-Chhinling wants to merge 9 commits intoapache:trunkfrom
Hean-Chhinling:trunk_YARN-11897
Open

YARN-11897: Make NMWebservices API backward-compatible after Jersey 2…#8227
Hean-Chhinling wants to merge 9 commits intoapache:trunkfrom
Hean-Chhinling:trunk_YARN-11897

Conversation

@Hean-Chhinling
Copy link
Contributor

@Hean-Chhinling Hean-Chhinling commented Feb 4, 2026

upgrade

Description of PR

Retrieving the logs for a running application fails as the logs CLI expects a JSON object as a response when calling the NM REST API, and it provides a JSON Array

Actual response with Jersey 2:

[
  {
    "containerLogsInfo": {
      "containerLogInfo": [
        {
          "fileName": "stderr",
          "fileSize": "0",
          "lastModifiedTime": "Wed Nov 12 13:26:19 +0000 2025"
        },
        ...
      ],
      "logAggregationType": "LOCAL",
      "containerId": "container_e80_1762940127665_0015_01_000003",
      "nodeId": "ccycloud-2.native-moxy4.root.comops.site:8041"
    }
  }
]

Expected response with Jersey 1:

{
  "containerLogsInfo": [
    {
      "containerLogInfo": {
        "fileName": "logfile1",
        "fileSize": "12",
        "lastModifiedTime": "Thu Nov 13 13:11:14 +0100 2025"
      },
      "logAggregationType": "LOCAL",
      "containerId": "container_0_0000_00_000000",
      "nodeId": "testhost.foo.com:8042"
    },
    {
      "containerLogInfo": {
        "fileName": "logfile1-aggregated",
        "fileSize": "23",
        "lastModifiedTime": "Thu Nov 13 13:11:14 +0100 2025"
      },
      "logAggregationType": "AGGREGATED",
      "containerId": "container_0_0000_00_000000",
      "nodeId": "testhost.foo.com_8042"
    }
  ]
}

How was this patch tested?

Tested on a cluster:

  • Logs CLI works for running/finished applications
  • UI v2 works when loading containers for a node

Response format for /ws/v1/node/containers/{container-id}/logs endpoint seems compatible with the Jersey 1 format comparing them with a nightly version:

JSON:

{
  "containerLogsInfo": [
    {
      "containerLogInfo": [
        {
          "fileName": "stderr",
          "fileSize": "0",
          "lastModifiedTime": "Wed Nov 19 08:40:57 +0000 2025"
        },
        {
          "fileName": "prelaunch.err",
          "fileSize": "0",
          "lastModifiedTime": "Wed Nov 19 08:40:57 +0000 2025"
        },
        {
          "fileName": "prelaunch.out",
          "fileSize": "70",
          "lastModifiedTime": "Wed Nov 19 08:40:57 +0000 2025"
        },
        {
          "fileName": "stdout",
          "fileSize": "0",
          "lastModifiedTime": "Wed Nov 19 08:40:57 +0000 2025"
        }
      ],
      "logAggregationType": "LOCAL",
      "containerId": "container_e87_1763463022925_0008_01_000003",
      "nodeId": "ccycloud-2.native-moxy4.root.comops.site:8041"
    }
  ]
}

XML:

<containerLogsInfoes>
   <containerLogsInfo>
      <containerLogInfo>
         <fileName>stderr</fileName>
         <fileSize>0</fileSize>
         <lastModifiedTime>Wed Nov 19 14:43:16 +0000 2025</lastModifiedTime>
      </containerLogInfo>
      <containerLogInfo>
         <fileName>prelaunch.err</fileName>
         <fileSize>0</fileSize>
         <lastModifiedTime>Wed Nov 19 14:43:16 +0000 2025</lastModifiedTime>
      </containerLogInfo>
      <containerLogInfo>
         <fileName>prelaunch.out</fileName>
         <fileSize>70</fileSize>
         <lastModifiedTime>Wed Nov 19 14:43:16 +0000 2025</lastModifiedTime>
      </containerLogInfo>
      <containerLogInfo>
         <fileName>stdout</fileName>
         <fileSize>0</fileSize>
         <lastModifiedTime>Wed Nov 19 14:43:16 +0000 2025</lastModifiedTime>
      </containerLogInfo>
      <logAggregationType>LOCAL</logAggregationType>
      <containerId>container_e88_1763555032540_0004_01_000003</containerId>
      <nodeId>ccycloud-2.native-moxy4.root.comops.site:8041</nodeId>
   </containerLogsInfo>
</containerLogsInfoes>

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 under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

AI Tooling

If an AI tool was used:

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 35s #8227 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/2/console
versions git=2.34.1
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@K0K0V0K
Copy link
Contributor

K0K0V0K commented Feb 10, 2026

Thanks @Hean-Chhinling for the fix!

LGTM!

@Hean-Chhinling
Copy link
Contributor Author

Why the Hadoop Yetus has not commented at all?

Seems to be a problem it could not detect that there is a new commit in the trunk_YARN-11897 branch.
Will try to push one more time and see.

17:33:31  Already up to date.
17:33:31  Already on 'trunk'
17:33:31  Your branch is up to date with 'origin/trunk'.
17:33:31  HEAD is now at fd18f0647dc HADOOP-19808. Jenkins switch from Debian 11 to Debian 13
17:33:32  ERROR: https://github.com/apache/hadoop/pull/8227 does not apply to trunk.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 11s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+0 🆗 xmllint 0m 1s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 9m 7s Maven dependency ordering for branch
+1 💚 mvninstall 46m 15s trunk passed
+1 💚 compile 2m 46s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 2m 49s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 4s trunk passed
+1 💚 mvnsite 2m 16s trunk passed
+1 💚 javadoc 2m 2s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 2m 0s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 56s trunk passed
+1 💚 shadedclient 34m 56s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 46s the patch passed
+1 💚 compile 2m 17s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 2m 17s the patch passed
+1 💚 compile 2m 23s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 2m 23s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 1m 34s /results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 4 new + 21 unchanged - 0 fixed = 25 total (was 21)
+1 💚 mvnsite 1m 23s the patch passed
+1 💚 javadoc 1m 8s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 8s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 26s the patch passed
+1 💚 shadedclient 34m 3s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 4m 32s hadoop-yarn-server-common in the patch passed.
-1 ❌ unit 26m 25s /patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager.txt hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 37s /results-asflicense.txt The patch generated 3 ASF License warnings.
193m 15s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/3/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux 4f42018f3106 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / a666c4b
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/3/testReport/
Max. process+thread count 576 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/3/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 13m 4s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 8m 34s Maven dependency ordering for branch
-1 ❌ mvninstall 36m 39s /branch-mvninstall-root.txt root in trunk failed.
+1 💚 compile 2m 45s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 2m 45s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 1s trunk passed
+1 💚 mvnsite 2m 18s trunk passed
+1 💚 javadoc 2m 12s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 2m 0s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 53s trunk passed
+1 💚 shadedclient 29m 1s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 30s Maven dependency ordering for patch
+1 💚 mvninstall 1m 45s the patch passed
+1 💚 compile 2m 13s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 2m 13s the patch passed
+1 💚 compile 2m 17s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 2m 17s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 1m 28s the patch passed
+1 💚 mvnsite 1m 25s the patch passed
+1 💚 javadoc 1m 11s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 10s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 16s the patch passed
+1 💚 shadedclient 27m 57s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 4m 12s hadoop-yarn-server-common in the patch passed.
-1 ❌ unit 26m 20s /patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager.txt hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 36s /results-asflicense.txt The patch generated 2 ASF License warnings.
181m 40s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/4/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux 0b4a112eecf8 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 3ffa8c9
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/4/testReport/
Max. process+thread count 613 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/4/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 13m 1s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 8m 12s Maven dependency ordering for branch
+1 💚 mvninstall 38m 3s trunk passed
+1 💚 compile 2m 40s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 2m 46s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 4s trunk passed
+1 💚 mvnsite 2m 21s trunk passed
+1 💚 javadoc 2m 10s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 2m 8s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 54s trunk passed
+1 💚 shadedclient 29m 1s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 45s the patch passed
+1 💚 compile 2m 13s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 2m 13s the patch passed
+1 💚 compile 2m 14s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 2m 14s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 1m 32s the patch passed
+1 💚 mvnsite 1m 22s the patch passed
+1 💚 javadoc 1m 11s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 8s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 17s the patch passed
+1 💚 shadedclient 27m 48s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 4m 10s hadoop-yarn-server-common in the patch passed.
-1 ❌ unit 26m 14s /patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager.txt hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 36s /results-asflicense.txt The patch generated 2 ASF License warnings.
182m 44s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/5/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux 5d4898a83fd2 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 048dd44
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/5/testReport/
Max. process+thread count 634 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/5/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 36s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 10m 35s Maven dependency ordering for branch
+1 💚 mvninstall 37m 42s trunk passed
+1 💚 compile 2m 41s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 2m 45s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 3s trunk passed
+1 💚 mvnsite 2m 18s trunk passed
+1 💚 javadoc 2m 9s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 2m 9s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 54s trunk passed
+1 💚 shadedclient 28m 53s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 43s the patch passed
+1 💚 compile 2m 13s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 2m 13s the patch passed
+1 💚 compile 2m 16s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 2m 16s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 1m 30s the patch passed
+1 💚 mvnsite 1m 21s the patch passed
+1 💚 javadoc 1m 10s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 11s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 15s the patch passed
+1 💚 shadedclient 27m 53s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 4m 8s hadoop-yarn-server-common in the patch passed.
-1 ❌ unit 26m 9s /patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager.txt hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 37s /results-asflicense.txt The patch generated 2 ASF License warnings.
172m 6s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/6/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux 323a4995775c 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 0a3215f
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/6/testReport/
Max. process+thread count 769 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/6/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@Hean-Chhinling
Copy link
Contributor Author

Tested running the tests locally and all pass:

OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended
[INFO] Tests run: 19, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.254 s -- in org.apache.hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 19, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  11.960 s
[INFO] Finished at: 2026-02-19T10:08:34+01:00
[INFO] ------------------------------------------------------------------------

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 6m 56s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 8m 48s Maven dependency ordering for branch
+1 💚 mvninstall 21m 37s trunk passed
+1 💚 compile 1m 28s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 1m 26s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 1m 11s trunk passed
+1 💚 mvnsite 1m 24s trunk passed
+1 💚 javadoc 1m 16s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 17s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 2m 21s trunk passed
+1 💚 shadedclient 15m 55s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 14s Maven dependency ordering for patch
+1 💚 mvninstall 1m 4s the patch passed
+1 💚 compile 1m 12s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 1m 12s the patch passed
+1 💚 compile 1m 20s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 1m 20s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 0m 48s the patch passed
+1 💚 mvnsite 0m 51s the patch passed
+1 💚 javadoc 0m 45s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 0m 44s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 1m 58s the patch passed
+1 💚 shadedclient 15m 19s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 2m 59s hadoop-yarn-server-common in the patch passed.
-1 ❌ unit 22m 48s /patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager.txt hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 24s /results-asflicense.txt The patch generated 2 ASF License warnings.
115m 37s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.webapp.TestNMWebServices
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/7/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux 0bda53ff3b29 5.15.0-141-generic #151-Ubuntu SMP Sun May 18 21:35:19 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 33dc58d
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/7/testReport/
Max. process+thread count 639 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/7/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@Hean-Chhinling
Copy link
Contributor Author

Hean-Chhinling commented Feb 19, 2026

Very strange the test pass when running locally but failed when running on CI:

[ERROR] org.apache.hadoop.yarn.server.nodemanager.webapp.TestNMWebServices.testContainerLogsWithNewAPI -- Time elapsed: 0.488 s <<< FAILURE!
org.opentest4j.AssertionFailedError: 

Expecting:
 <"logfile2">
to be equal to:
 <"logfile1">
but was not.

Update: This is because the file style does not guarantee order. Those updating the test to only use one file like previously.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 31s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 9m 22s Maven dependency ordering for branch
+1 💚 mvninstall 38m 55s trunk passed
+1 💚 compile 2m 42s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 2m 47s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 1s trunk passed
+1 💚 mvnsite 2m 22s trunk passed
+1 💚 javadoc 2m 13s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 2m 8s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 56s trunk passed
+1 💚 shadedclient 28m 46s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 30s Maven dependency ordering for patch
+1 💚 mvninstall 1m 45s the patch passed
+1 💚 compile 2m 11s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 2m 11s the patch passed
+1 💚 compile 2m 17s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 2m 17s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 1m 30s the patch passed
+1 💚 mvnsite 1m 22s the patch passed
+1 💚 javadoc 1m 11s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 10s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 3m 16s the patch passed
+1 💚 shadedclient 27m 48s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 4m 12s hadoop-yarn-server-common in the patch passed.
+1 💚 unit 26m 25s hadoop-yarn-server-nodemanager in the patch passed.
-1 ❌ asflicense 0m 37s /results-asflicense.txt The patch generated 2 ASF License warnings.
172m 3s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/8/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux df11d7be04ad 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 765a434
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/8/testReport/
Max. process+thread count 615 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/8/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 22s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 9m 27s Maven dependency ordering for branch
+1 💚 mvninstall 21m 17s trunk passed
+1 💚 compile 1m 28s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 1m 32s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 1m 6s trunk passed
+1 💚 mvnsite 1m 27s trunk passed
+1 💚 javadoc 1m 21s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 21s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 2m 18s trunk passed
+1 💚 shadedclient 15m 44s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 16s Maven dependency ordering for patch
+1 💚 mvninstall 1m 4s the patch passed
+1 💚 compile 1m 12s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 1m 12s the patch passed
+1 💚 compile 1m 18s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 1m 18s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 0m 49s the patch passed
+1 💚 mvnsite 0m 50s the patch passed
+1 💚 javadoc 0m 44s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 0m 46s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 1m 59s the patch passed
+1 💚 shadedclient 15m 17s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 2m 59s hadoop-yarn-server-common in the patch passed.
+1 💚 unit 22m 44s hadoop-yarn-server-nodemanager in the patch passed.
+1 💚 asflicense 0m 24s The patch does not generate ASF License warnings.
109m 6s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/9/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux dee766223e61 5.15.0-141-generic #151-Ubuntu SMP Sun May 18 21:35:19 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 6d6cb91
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/9/testReport/
Max. process+thread count 603 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/9/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@Hean-Chhinling
Copy link
Contributor Author

Hello @K0K0V0K @p-szucs @slfan1989 !
Could you help review this PR for backward compatibility with Jersey1?

@slfan1989
Copy link
Contributor

@K0K0V0K Mind doing a first-pass review on this PR? I’m a bit bandwidth-constrained lately. Thank you very much!

Copy link
Contributor

@K0K0V0K K0K0V0K left a comment

Choose a reason for hiding this comment

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

Thanks @Hean-Chhinling for the change this seems mostly fine by me.
I just had a one bigger question left.

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 8m 31s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 4 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 2m 16s Maven dependency ordering for branch
+1 💚 mvninstall 27m 43s trunk passed
+1 💚 compile 1m 28s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 1m 28s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 1m 10s trunk passed
+1 💚 mvnsite 1m 27s trunk passed
+1 💚 javadoc 1m 20s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 23s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 2m 16s trunk passed
+1 💚 shadedclient 16m 2s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 16s Maven dependency ordering for patch
+1 💚 mvninstall 1m 3s the patch passed
+1 💚 compile 1m 12s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javac 1m 12s the patch passed
+1 💚 compile 1m 14s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 javac 1m 14s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 0m 51s /results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 9 new + 21 unchanged - 0 fixed = 30 total (was 21)
+1 💚 mvnsite 0m 51s the patch passed
+1 💚 javadoc 0m 47s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 0m 47s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 spotbugs 2m 2s the patch passed
+1 💚 shadedclient 14m 52s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 3m 3s hadoop-yarn-server-common in the patch passed.
+1 💚 unit 22m 40s hadoop-yarn-server-nodemanager in the patch passed.
+1 💚 asflicense 0m 24s The patch does not generate ASF License warnings.
116m 39s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/10/artifact/out/Dockerfile
GITHUB PR #8227
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint
uname Linux c444efdaca52 5.15.0-141-generic #151-Ubuntu SMP Sun May 18 21:35:19 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 8615bf8
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/10/testReport/
Max. process+thread count 634 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/10/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@Hean-Chhinling
Copy link
Contributor Author

@K0K0V0K If it is all looks good to you, could I get a tick on this PR?
So that we could move to the next round of review from @slfan1989

@K0K0V0K
Copy link
Contributor

K0K0V0K commented Feb 26, 2026

Thanks Ling!

This looks fine to me.
LGTM!

@K0K0V0K
Copy link
Contributor

K0K0V0K commented Feb 26, 2026

One minor thing i just saw:
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/10/artifact/out/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt

may i ask you to fix these styles?

@slfan1989
Copy link
Contributor

@K0K0V0K Thanks for your proactive review and suggestions; @Hean-Chhinling Thanks for your contributions and support. I’ll complete the review of this PR tomorrow.

@slfan1989 slfan1989 requested review from slfan1989 and removed request for K0K0V0K February 27, 2026 01:58
@Hean-Chhinling
Copy link
Contributor Author

One minor thing i just saw: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8227/10/artifact/out/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt

may i ask you to fix these styles?

Thank you! I fixed it in my new commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants