Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-19 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2601063200

   @steveloughran @Hexiaoqiao 
   
   I believe the overall upgrade is currently in line with expectations. I have 
reviewed the overall build report, and there are no issues with JDK 8. However, 
with JDK 11, there is a reported issue regarding the missing 
`com.sun.xml.bind.v2.ContextFactory`. I will continue to follow up and work on 
fixing it.
   
   cc: @aajisaka @ayushtkn @pjfanning @virajjasani


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2597346603

   @Hexiaoqiao @steveloughran Thank you for approving this PR! If any issues 
arise in the future, I will continue to follow up. However, I remain optimistic 
since we haven't changed the core logic, and the primary changes are related to 
testing. I will also keep optimizing the code to prepare for a future upgrade 
to Jersey 3, and I already have some ideas for that.
   
   I plan to merge this PR in 2 working days.
   
   I will add the following members as co-authors in the commit to express my 
gratitude.
   
   @aajisaka @@virajjasani These two members provided the initial version of 
HADOOP-15984.
   @ayushtkn provided a customized version of yetus, which has made my overall 
testing process much easier.
   @pjfanning have submitted multiple PRs previously, effectively reducing the 
scope of changes for HADOOP-15984.
   @steveloughran and @Hexiaoqiao helped review the code and provided many 
valuable suggestions. I don't think I would have been able to push through 
without your encouragement.
   
   At the same time, I hope our JDK 17 upgrade goes smoothly. After this PR, we 
plan to submit a large-scale PR to upgrade from JUnit4 to JUnit5. We have 
already prepared for this locally.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


Hexiaoqiao commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2597313845

   Thanks all for your great works. make sense to. +1 from my side.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2597215502

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 30s |  |  
https://github.com/apache/hadoop/pull/7019 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-7019/47/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1919420354


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   @Hexiaoqiao @steveloughran In the latest commit, I have rolled back the 
Jenkinsfile commit, which may cause the next build to not complete 
successfully, as the current CI cannot compile code with more than 10,000 
lines. However, this is expected, so please don't worry.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2597203683

   @steveloughran @Hexiaoqiao @pjfanning Thank you all for reviewing this PR! 
We have compiled it multiple times, and currently, the failing unit tests seem 
unrelated to our changes. Thanks to @pjfanning  for the suggestions—they have 
been really helpful. Based on the current progress, I plan to merge this PR 
first and then assess if there are any other improvements we can submit in 
future PRs.
   
   Additionally, I will revert the changes to `dev-support/Jenkinsfile` in the 
next commit. I also want to thank @ayushtkn for providing the Yetus package to 
help me complete this PR, as our current CI environment cannot handle 
compilations for code over 10,000 lines. Therefore, the next commit may not 
produce a full compilation result, but please rest assured, it won’t affect the 
overall progress.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2596347120

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/46/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 49s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m 34s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m  3s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m 35s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   5m 40s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m 18s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 54s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 52s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 18s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 21s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  66m 27s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  66m 51s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 44s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  81m 23s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 15s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m 15s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m  5s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  16m  5s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   5m  3s |  |  root: The patch generated 
0 new + 1605 unchanged - 100 fixed = 1605 total (was 1705)  |
   | +1 :green_heart: |  mvnsite  |  19m 21s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 50s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0 new + 5541 unchanged - 
168 fixed = 5541 total (was 5709)  |
   | +1 :green_heart: |  javadoc  |   7m 48s |  |  
root-jdkPrivateBuild-1.

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-16 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2596312043

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 34s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  8s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  8s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/45/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 45s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m 33s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 36s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m 43s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   5m  5s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m 52s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  10m  5s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 45s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 17s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  65m 12s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  65m 35s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 45s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  82m 58s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m 30s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  17m 30s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 12s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  16m 12s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  1s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   4m 39s |  |  root: The patch generated 
0 new + 1605 unchanged - 100 fixed = 1605 total (was 1705)  |
   | +1 :green_heart: |  mvnsite  |  16m 45s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |  10m 10s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0 new + 5541 unchanged - 
168 fixed = 5541 total (was 5709)  |
   | +1 :green_heart: |  javadoc  |   7m 51s |  |  
root-jdkPrivateBuild-1.

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-15 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1917570435


##
dev-support/Jenkinsfile:
##
@@ -71,7 +71,7 @@ pipeline {
 checkout([
 $class: 'GitSCM',
 branches: [[name: "${env.YETUS_VERSION}"]],
-userRemoteConfigs: [[ url: 
'https://github.com/apache/yetus.git']]]
+userRemoteConfigs: [[ url: 
'https://github.com/ayushtkn/yetus.git']]]

Review Comment:
   @Hexiaoqiao Thank you very much for your message! I have carefully checked 
the agreement for the new package we introduced, as follows:
   
   > Apache License Version 2.0
   
   ```
   org.glassfish.jersey.core:jersey-common:2.46
   org.glassfish.jersey.core:jersey-server:2.46
   org.glassfish.jersey.inject:jersey-hk2:2.46
   org.glassfish.jersey.core:jersey-client:2.46
   org.glassfish.jersey.test-framework:jersey-test-framework-core:2.46
   
org.glassfish.jersey.test-framework.providers:jersey-test-framework-provider-grizzly2:2.46
   org.glassfish.jersey.containers:jersey-container-servlet:2.46
   org.glassfish.jersey.containers:jersey-container-servlet-core:2.46
   org.glassfish.jersey.media:jersey-media-json-jettison:2.46
   org.glassfish.jersey.media:jersey-media-jaxb:2.46
   net.jodah:failsafe:2.4.4
   ```
   
   > Eclipse Distribution License (EDL) 1.0
   
   ```
   jakarta.xml.bind:jakarta.xml.bind-api:2.3.3
   ```
   
   > Eclipse Public License (EPL) 2.0
   
   ```
   jakarta.ws.rs-api:jakarta.ws.rs-api:2.1.6
   jakarta.servlet.jsp:jakarta.servlet.jsp-api:2.3.6
   jakarta.servlet:jakarta.servlet-api:4.0.4
   ```
   
   cc: @pjfanning 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-15 Thread via GitHub


Hexiaoqiao commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1916609008


##
dev-support/Jenkinsfile:
##
@@ -71,7 +71,7 @@ pipeline {
 checkout([
 $class: 'GitSCM',
 branches: [[name: "${env.YETUS_VERSION}"]],
-userRemoteConfigs: [[ url: 
'https://github.com/apache/yetus.git']]]
+userRemoteConfigs: [[ url: 
'https://github.com/ayushtkn/yetus.git']]]

Review Comment:
   It is better to link Hadoop repo rather than Ayushtkn's personal repo.



##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   It is OK to located this package to Apache License or MIT if it is dual 
licensed IMO, if we could not make sure, maybe refer Spark 
https://github.com/apache/spark/blob/master/LICENSE-binary and write it under 
Apache License will not be wrong. Thanks.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-15 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2592760131

   > I think we should merge this as is and make outstanding issues followups.
   > 
   > We will see problems, just for different deployments/test runs, so let's 
just encourage testing with the goal of finding those corner cases.
   > 
   > We need to do this for java17, so let's get it in and see what those 
corner cases are
   
   Many thanks to @steveloughran for the comment!
   
   Here’s a summary of the current situation: all compilations have passed. 
   
   The reported unit test failures include one that often times out 
(TestDecommission, which frequently appears in other PRs as well) and another 
that might be an occasional runtime failure (TestTimelineAuthFilterForV2), but 
it passes locally for me.
   
   I have checked the checkstyle and imports for all classes, and they should 
meet the expected standards. A special thanks to @ayushtkn for providing a 
Yetus package that has been helpful for continuous debugging. Before merging 
this PR, I will roll back the changes to the `dev-support/Jenkinsfile`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-15 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1916511634


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   @steveloughran Thank you for the acknowledgment! If there are no further 
comments, I plan to merge this PR in two days. After multiple rounds of 
testing, everything is performing as expected. I’ve personally debugged each 
unit test improvement to ensure they work in most scenarios. Of course, there 
may still be some occasional issues with individual tests (though the 
likelihood is low), and we can continue refining them in the future.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-14 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2590742811

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/44/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 51s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 50s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 24s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m  9s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 35s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 20s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 26s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 33s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 20s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  60m 56s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  61m 19s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 45s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  79m 18s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 37s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 37s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 19s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m 19s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   4m 31s |  |  root: The patch generated 
0 new + 1605 unchanged - 100 fixed = 1605 total (was 1705)  |
   | +1 :green_heart: |  mvnsite  |  18m 31s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 17s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0 new + 5541 unchanged - 
168 fixed = 5541 total (was 5709)  |
   | +1 :green_heart: |  javadoc  |   7m 32s |  |  
root-jdkPrivateBuild-1.

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-14 Thread via GitHub


steveloughran commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2590595775

   I think we should merge this as is and make outstanding issues followups.
   
   We will see problems, just for different deployments/test runs, so let's 
just encourage testing with the goal of finding those corner cases. 
   
   We need to do this for java17, so let's get it in and see what those corner 
cases are


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-14 Thread via GitHub


steveloughran commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1915269542


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   I'm in the "merge this and then deal with the details" mode right now. Get 
this in and then we can start the real burndown for java17. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-13 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2588706048

   The compilation report contains the following error:
   
   ```
   ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-federation-balance: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-dynamometer-infra: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-archive-logs: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-gridmix: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-aws: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-azure: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-sls: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test (default-test) on 
project hadoop-azure-datalake: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M1:test failed: 
java.lang.NoClassDefFoundError: 
org/junit/platform/launcher/core/LauncherFactory: 
org.junit.platform.launcher.core.LauncherFactory -> [Help 1]
   ```
   
   I will add the following dependencies to the relevant module.
   
   ```
   
 org.junit.jupiter
 junit-jupiter-api
 test
   
   
 org.junit.jupiter
 junit-jupiter-engine
 test
   
   
 org.junit.jupiter
 junit-jupiter-params
 test
   
   
 org.junit.platform
 junit-platform-launcher
 test
   
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-13 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2587190596

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 34s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/43/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 17s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 34s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 34s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  14m 47s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 30s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  21m 29s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 34s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 20s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  61m 10s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  61m 32s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 47s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  76m 43s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 25s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 25s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 26s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m 26s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 33s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/43/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 1 new + 1605 unchanged - 100 fixed = 1606 total 
(was 1705)  |
   | +1 :green_heart: |  mvnsite  |  16m 18s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 22s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912574217


##
hadoop-project/pom.xml:
##
@@ -2147,6 +2067,68 @@
 cache-api
 ${cache.api.version}
   
+  
+  
+org.glassfish.jersey.containers
+jersey-container-servlet-core
+${jersey2.version}
+  
+  
+org.glassfish.jersey.containers
+jersey-container-servlet
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-client
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-common
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-server
+${jersey2.version}
+  
+  
+org.glassfish.jersey.media
+jersey-media-jaxb
+${jersey2.version}
+  
+  
+org.glassfish.jersey.inject
+jersey-hk2
+${jersey2.version}
+  
+  
+org.glassfish.jersey.test-framework
+jersey-test-framework-core
+${jersey2.version}
+  
+  
+org.glassfish.jersey.test-framework.providers
+jersey-test-framework-provider-grizzly2
+${jersey2.version}
+
+  
+javax.servlet
+javax.servlet-api
+  
+
+  
+  
+org.glassfish.jersey.media
+jersey-media-json-jettison

Review Comment:
   Thank you for your suggestion! After considering it, would it be possible to 
submit a new PR to address and optimize the Jettison issue after this PR is 
merged?
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585966046

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/42/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m 54s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  34m 31s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  16m  5s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   5m  1s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  18m 35s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  10m  4s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   8m  3s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 18s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  64m 33s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  64m 55s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 43s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  75m  8s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m 51s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  17m 51s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  3s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  17m  3s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  1s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 52s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/42/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 5 new + 1605 unchanged - 100 fixed = 1610 total 
(was 1705)  |
   | +1 :green_heart: |  mvnsite  |  19m 24s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 55s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585950017

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/41/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 181 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 28s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 32s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m  6s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  14m 50s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 27s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m 46s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  10m 19s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 45s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  67m 28s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  67m 52s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 43s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  73m  9s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  3s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  17m  3s |  |  the patch passed  |
   | -1 :x: |  compile  |  10m 23s | 
[/patch-compile-root-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/41/artifact/out/patch-compile-root-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt)
 |  root in the patch failed with JDK Private 
Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.  |
   | -1 :x: |  javac  |  10m 23s | 
[/patch-compile-root-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/41/artifact/out/patch-compile-root-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt)
 |  root in the patch failed with JDK Private 
Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912461961


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   I think we can wait for input from the other members, as I'm also unsure 
about the best way to improve it. Once we have everyone's feedback, we can 
decide on the changes together. It's a good sign that we've already started 
reviewing this PR.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


pjfanning commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912459732


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   I'm not sure if jersey 2 is dual licensed. Reading its license and notice 
makes it look like it is EPL-2.0 licensed by with some parts licensed under 1 
or more of the other 3 licenses.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912459162


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   Thank you very much for your reply! I will improve it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


pjfanning commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912457983


##
LICENSE-binary:
##
@@ -498,12 +498,16 @@ org.slf4j:slf4j-reload4j:1.7.36
 CDDL 1.1 + GPLv2 with classpath exception
 -
 
-com.github.pjfanning:jersey-json:1.22.0
-com.sun.jersey:jersey-client:1.19.4
-com.sun.jersey:jersey-core:1.19.4
-com.sun.jersey:jersey-guice:1.19.4
-com.sun.jersey:jersey-server:1.19.4
-com.sun.jersey:jersey-servlet:1.19.4
+org.glassfish.jersey.core:jersey-common:2.46

Review Comment:
   this looks like the wrong license group
   
   
https://mvnrepository.com/artifact/org.glassfish.jersey.core/jersey-common/2.46
   
   jersey 2 is licensed under Apache License (very important) and 3 other 
licenses.
   
   ```
   
   
   EPL 2.0
   http://www.eclipse.org/legal/epl-2.0
   repo
   Except for Guava, JSR-166 files, Dropwizard Monitoring inspired 
classes, ASM and Jackson JAX-RS Providers. See also 
https://github.com/eclipse-ee4j/jersey/blob/master/NOTICE.md
   
   
   The GNU General Public License (GPL), Version 2, With Classpath 
Exception
   https://www.gnu.org/software/classpath/license.html
   repo
   Except for Guava, and JSR-166 files. See also 
https://github.com/eclipse-ee4j/jersey/blob/master/NOTICE.md
   
   
   Apache License, 2.0
   http://www.apache.org/licenses/LICENSE-2.0.html
   repo
   Google Guava @ org.glassfish.jersey.internal.guava
   
   
   Public Domain
   https://creativecommons.org/publicdomain/zero/1.0/
   repo
   JSR-166 Extension to JEP 266 @ 
org.glassfish.jersey.internal.jsr166
   
   
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


pjfanning commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1912456790


##
hadoop-project/pom.xml:
##
@@ -2147,6 +2067,68 @@
 cache-api
 ${cache.api.version}
   
+  
+  
+org.glassfish.jersey.containers
+jersey-container-servlet-core
+${jersey2.version}
+  
+  
+org.glassfish.jersey.containers
+jersey-container-servlet
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-client
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-common
+${jersey2.version}
+  
+  
+org.glassfish.jersey.core
+jersey-server
+${jersey2.version}
+  
+  
+org.glassfish.jersey.media
+jersey-media-jaxb
+${jersey2.version}
+  
+  
+org.glassfish.jersey.inject
+jersey-hk2
+${jersey2.version}
+  
+  
+org.glassfish.jersey.test-framework
+jersey-test-framework-core
+${jersey2.version}
+  
+  
+org.glassfish.jersey.test-framework.providers
+jersey-test-framework-provider-grizzly2
+${jersey2.version}
+
+  
+javax.servlet
+javax.servlet-api
+  
+
+  
+  
+org.glassfish.jersey.media
+jersey-media-json-jettison

Review Comment:
   * jersey-media-json-jettison uses a vulnerable version of jettison
   * 
https://mvnrepository.com/artifact/org.glassfish.jersey.media/jersey-media-json-jettison/2.46
   * would it be possible to exclude the jettison transitive dependency and 
explicitly import the newer version of jettison?
   * I think hadoop already has a jettison dependency set up
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585735442

   @ayushtkn @aajisaka @virajjasani @steveloughran @Hexiaoqiao @pjfanning 
   
   We have mostly completed the preparation for the Jersey version upgrade. In 
this upgrade, we will be moving to Jersey 2.46. I have finished the basic 
functionality testing and the necessary changes to the JUnit tests. 
   
   I would appreciate it if you could review the code and provide any feedback. 
I have outlined the overall upgrade process above, and I’m available to answer 
any questions if needed.
   
   If this PR is approved, I will continue to accelerate the work on Hadoop's 
support for JDK 17.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-12 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585703680

   I have completed the review of all modules and we can say that the upgrade 
from Jersey 1.x to Jersey 2.x is essentially complete. We can now begin the PR 
review. There are only a few minor questions, but I believe they should not be 
blockers. I will summarize and explain them in detail below:
   
   > Background
   
   We are upgrading Hadoop to support JDK 17, and one of the biggest challenges 
is upgrading Jersey from 1.x to 2.x. There are many differences between Jersey 
1.x and Jersey 2.x, and they are almost entirely unrelated, especially since 
Jersey 2.x has undergone a significant internal refactor and now includes its 
own dependency injection framework, which makes integration with Guice very 
difficult.
   
   I noticed that many community members had previously attempted to integrate 
Guice into Jersey 2.x, but with limited success. At the beginning of this PR, I 
also tried this approach, but it didn’t progress smoothly. After careful 
consideration, I decided to move away from Guice and directly use Jersey 2.x's 
own dependency injection framework.
   
   I initially tested this approach in the NodeManager module and confirmed its 
feasibility. I then applied this solution to other modules and validated that 
it works as expected.
   
   I have created a simple upgrade diagram, as shown below:
   
   https://github.com/user-attachments/assets/d36673aa-543e-4f1a-82e2-d7aa36eb3afd";
 />
   
   > Upgrade process
   
   Step1. The way Jetty is injected has been adjusted. 
   
   We now rely on Jersey 2's `ResourceConfig` for package scanning to complete 
dependency injection. We can refer to the following code:
   
   
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java
   
   ```
   /**
  * Add a Jersey resource package.
  * @param packageName The Java package name containing the Jersey resource.
  * @param pathSpec The path spec for the servlet
  * @param params properties and features for ResourceConfig
  */
 public void addJerseyResourcePackage(final String packageName,
 final String pathSpec, Map params) {
   LOG.info("addJerseyResourcePackage: packageName = {}, pathSpec = {}.",
   packageName, pathSpec);
   final ResourceConfig config = new ResourceConfig().packages(packageName);
   final ServletHolder sh = new ServletHolder(new ServletContainer(config));
   for (Map.Entry entry : params.entrySet()) {
 sh.setInitParameter(entry.getKey(), entry.getValue());
   }
   webAppContext.addServlet(sh, pathSpec);
 }
   
 /**
  * Add a Jersey resource config.
  * @param config The Jersey ResourceConfig to be registered.
  * @param pathSpec The path spec for the servlet
  * @param params properties and features for ResourceConfig
  */
 public void addJerseyResourceConfig(final ResourceConfig config,
 final String pathSpec, Map params) {
   LOG.info("addJerseyResourceConfig: pathSpec = {}.", pathSpec);
   final ServletHolder sh = new ServletHolder(new ServletContainer(config));
   if (params != null) {
 for (Map.Entry entry : params.entrySet()) {
   sh.setInitParameter(entry.getKey(), entry.getValue());
 }
   }
   webAppContext.addServlet(sh, pathSpec);
 }
   ```
   
   Step2.  Modify the way WebServices are injected in each module.
   
   We will take ResourceManager as an example.
   
   
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
   
   - @Inject, @Singleton, and @Name are all from the standard javax 
annotations, not from Guice annotations.
   
   ```
 @Inject
 public RMWebServices(final @javax.inject.Named("rm") ResourceManager rm,
 @javax.inject.Named("conf") Configuration conf) {
   // don't inject, always take appBaseRoot from RM.
   super(null);
   this.rm = rm;
   this.conf = conf;
   isCentralizedNodeLabelConfiguration =
   YarnConfiguration.isCentralizedNodeLabelConfiguration(conf);
   this.filterAppsByUser  = conf.getBoolean(
   YarnConfiguration.FILTER_ENTITY_LIST_BY_USER,
   YarnConfiguration.DEFAULT_DISPLAY_APPS_FOR_LOGGED_IN_USER);
 .
   }
 }
   ```
   
   
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebApp.java
   
   - When the ResourceManager starts, the instances of RM and Conf are directly 
injected into RMWebServices via Jersey2.
   
   ```
 public ResourceConfig resourceConfig() {
   ResourceConfig config = new ResourceConfig();
   config.packages("org.apache.hadoop.yarn.server.resourcemanager.webapp");
   config.register(new JerseyBinder()

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-11 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585477630

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/40/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 182 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m 59s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 24s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m  0s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 34s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 14s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 22s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 36s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  60m 30s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  60m 53s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 46s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  74m 53s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 13s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 13s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  14m 59s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  14m 59s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 25s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/40/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 8 new + 1605 unchanged - 100 fixed = 1613 total 
(was 1705)  |
   | +1 :green_heart: |  mvnsite  |  18m 19s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 17s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-10 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2585049475

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/39/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 183 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 18s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 21s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 32s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m  4s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 34s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 25s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 24s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 33s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 21s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  61m 23s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  61m 45s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 46s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  70m 13s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 16s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 16s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 18s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m 18s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 28s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/39/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 1 new + 1604 unchanged - 105 fixed = 1605 total 
(was 1709)  |
   | +1 :green_heart: |  mvnsite  |  18m 30s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 21s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-09 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2580869236

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 31s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/38/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 175 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 36s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 53s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 38s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  17m 13s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 33s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 37s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 43s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   8m  8s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  65m 10s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  65m 32s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 43s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  69m 39s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 19s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m 19s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  2s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  17m  2s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  1s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 39s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/38/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 23 new + 1604 unchanged - 105 fixed = 1627 total 
(was 1709)  |
   | +1 :green_heart: |  mvnsite  |  16m 32s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 35s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-09 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2580866300

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 42s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/37/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 175 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 33s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 44s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 53s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  17m 14s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 33s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 26s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 45s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 58s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 17s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  64m 32s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  64m 52s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 43s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  69m 49s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 14s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m 14s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  1s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  17m  1s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 26s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/37/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 23 new + 1604 unchanged - 105 fixed = 1627 total 
(was 1709)  |
   | +1 :green_heart: |  mvnsite  |  19m  0s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 42s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-06 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2573993466

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/36/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 175 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 46s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 30s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 31s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  16m 33s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 38s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m  2s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  10m  6s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 33s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  63m 22s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  63m 50s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 44s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  74m 30s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 38s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m 38s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  4s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  17m  4s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 38s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/36/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 43 new + 1605 unchanged - 104 fixed = 1648 total 
(was 1709)  |
   | +1 :green_heart: |  mvnsite  |  19m 18s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 50s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-06 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2573870619

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/35/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 170 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 25s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 18s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 26s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  14m 52s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 21s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m 55s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 20s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 27s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 21s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 21s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 21s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  67m 21s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  67m 43s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 42s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  72m 32s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m 33s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m 33s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 16s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  16m 16s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 37s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/35/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 90 new + 1607 unchanged - 102 fixed = 1697 total 
(was 1709)  |
   | +1 :green_heart: |  mvnsite  |  19m 15s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 19s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2571805479

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/34/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 160 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m 44s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  37m 19s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 49s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  17m  2s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   5m 26s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  18m  2s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  10m 40s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 54s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 18s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 23s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 22s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  68m 21s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  68m 42s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 43s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  74m 22s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 23s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 23s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m  2s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m  2s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 24s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/34/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 141 new + 1608 unchanged - 102 fixed = 1749 total 
(was 1710)  |
   | +1 :green_heart: |  mvnsite  |  18m 15s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 15s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2571662675

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 34s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/33/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 160 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m 55s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m 56s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 17s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m 19s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 35s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m  5s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 22s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 34s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 19s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests
 no spotbugs output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  61m  1s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  61m 24s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 45s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  69m 45s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 28s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 28s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 32s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m 31s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 27s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/33/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 137 new + 1608 unchanged - 102 fixed = 1745 total 
(was 1710)  |
   | +1 :green_heart: |  mvnsite  |  18m  0s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 12s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1903240972


##
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/webapp/TestAMWebServices.java:
##
@@ -68,215 +68,212 @@
  */
 public class TestAMWebServices extends JerseyTestBase {
 
-  private static Configuration conf = new Configuration();
+  private static final Configuration CONF = new Configuration();
   private static MockAppContext appContext;
 
-  private static class WebServletModule extends ServletModule {
-@Override
-protected void configureServlets() {
+  @Override
+  protected Application configure() {
+ResourceConfig config = new ResourceConfig();
+config.register(new JerseyBinder());
+config.register(AMWebServices.class);
+config.register(GenericExceptionHandler.class);
+config.register(new JettisonFeature()).register(JAXBContextResolver.class);
+return config;
+  }
 
+  private static class JerseyBinder extends AbstractBinder {
+@Override
+protected void configure() {
   appContext = new MockAppContext(0, 1, 1, 1);
   appContext.setBlacklistedNodes(Sets.newHashSet("badnode1", "badnode2"));
-
-  bind(JAXBContextResolver.class);
-  bind(AMWebServices.class);
-  bind(GenericExceptionHandler.class);
-  bind(AppContext.class).toInstance(appContext);
-  bind(Configuration.class).toInstance(conf);
-
-  serve("/*").with(GuiceContainer.class);
+  App app = new App(appContext);
+  bind(appContext).to(AppContext.class).named("am");
+  bind(app).to(App.class).named("app");
+  bind(CONF).to(Configuration.class).named("conf");
+  final HttpServletResponse response = mock(HttpServletResponse.class);
+  bind(response).to(HttpServletResponse.class);
 }
   }
 
-  static {
-GuiceServletConfig.setInjector(
-Guice.createInjector(new WebServletModule()));
-  }
-
-  @Before
   @Override
   public void setUp() throws Exception {

Review Comment:
   I will prepare a comprehensive document to explain all my improvement ideas. 
The part you mentioned will be included in the document.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1903240788


##
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/JAXBContextResolver.java:
##
@@ -50,6 +48,7 @@
 import org.apache.hadoop.mapreduce.v2.app.webapp.dao.TaskInfo;
 import org.apache.hadoop.mapreduce.v2.app.webapp.dao.TasksInfo;
 import org.apache.hadoop.yarn.webapp.RemoteExceptionData;
+import org.glassfish.jersey.jettison.JettisonJaxbContext;

Review Comment:
   This part has been fixed. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1903240456


##
hadoop-client-modules/hadoop-client-runtime/pom.xml:
##
@@ -157,8 +158,24 @@
   org.bouncycastle:*
   
   org.xerial.snappy:*
-  
-  org.jetbrains.kotlin:*

Review Comment:
   I have completed the fixes in the latest version.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-05 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2571575703

   HADOOP-15984 is nearly ready. I am currently doing a final round of code 
review to ensure there are no checkstyle issues and to make sure the backport 
is as simple as possible.
   
   The following modules have been checked.
   ```
   hadoop-auth
   hadoop-auth-examples
   hadoop-azure
   hadoop-client-check-invariants
   hadoop-client-check-test-invariants
   hadoop-client-integration-tests
   hadoop-client-minicluster
   hadoop-client-runtime
   hadoop-cloud-storage
   hadoop-common
   hadoop-hdfs
   hadoop-hdfs-client
   hadoop-hdfs-httpfs
   hadoop-hdfs-nfs
   hadoop-hdfs-rbf
   hadoop-huaweicloud
   hadoop-kms
   hadoop-mapreduce-client
   hadoop-mapreduce-client-app
   hadoop-mapreduce-client-hs
   hadoop-mapreduce-client-shuffle
   hadoop-mapreduce-client-example
   hadoop-nfs 
   hadoop-project
   hadoop-resourceestimator
   hadoop-yarn-api
   hadoop-yarn-application-catalog-webapp
   hadoop-yarn-application-distributedshell
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-04 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2571416740

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 34s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  7s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  7s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  7s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  7s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/32/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 161 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   6m  8s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 10s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m  1s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 25s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  19m  9s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   9m 22s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javadoc  |   7m 35s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +0 :ok: |  spotbugs  |   0m 20s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  61m  7s |  |  branch has no errors 
when building and testing our client artifacts.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 46s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  69m 30s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 20s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 20s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 10s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m 10s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 30s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/32/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 130 new + 1651 unchanged - 102 fixed = 1781 total 
(was 1753)  |
   | +1 :green_heart: |  mvnsite  |  18m  8s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   9m 17s |  |  
root-jdkUbuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 generated 0 new + 5546 unchanged - 
163 fixed = 5546 total (was 5709)  |
   | +1 :green_heart: |  javadoc  |   7m 35s |  |  
root-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga with JDK Private 
Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga generated 0 new + 1434 unchanged 
- 4 fixed = 1434 total (was 1438)  |
   | +0 :ok: |  spotbugs  |   0m 20s |  |  ha

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-03 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2569711701

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/31/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 159 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   7m 11s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m  7s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 34s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  17m  2s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 32s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  34m 58s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  31m 29s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | -1 :x: |  javadoc  |   0m 23s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/31/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  33m 36s |  |  branch has no errors 
when building and testing our client artifacts.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 46s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  38m 47s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m 54s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  17m 54s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  1s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  17m  1s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   4m 49s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/31/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 128 new + 1646 unchanged - 107 fixed = 1774 total 
(was 1753)  |
   | +

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2025-01-03 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2569711240

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  11m 47s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/30/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |   7m 45s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 18s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 42s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  17m  5s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 38s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  35m 15s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  31m 30s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | -1 :x: |  javadoc  |   0m 24s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/30/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  33m 30s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  33m 57s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 44s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  39m 27s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  18m  2s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  18m  2s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 59s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  16m 59s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   4m 22s |  |  root: The patch generated 
0 new + 1638 unchange

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-12-08 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2527141707

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  11m 32s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/28/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 54s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  30m 27s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  16m 16s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  compile  |  15m  5s |  |  trunk passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  checkstyle  |   4m 30s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  39m 10s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  35m 58s |  |  trunk passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | -1 :x: |  javadoc  |   0m 30s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/28/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga.  |
   | +0 :ok: |  spotbugs  |   0m 43s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 40s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  33m 16s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  33m 43s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 25s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  39m 30s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 35s |  |  the patch passed with JDK 
Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04  |
   | +1 :green_heart: |  javac  |  16m 35s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m  6s |  |  the patch passed with JDK 
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga  |
   | +1 :green_heart: |  javac  |  15m  6s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   4m 24s |  |  root: The patch generated 
0 new + 1638 unchange

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-11-05 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2457949679

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  12m 53s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/26/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  15m 36s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  32m  5s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 26s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  compile  |  16m  2s |  |  trunk passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  checkstyle  |   4m 37s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  40m  7s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  36m 45s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | -1 :x: |  javadoc  |   0m 31s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/26/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05.  |
   | +0 :ok: |  spotbugs  |   0m 40s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  35m  9s |  |  branch has no errors 
when building and testing our client artifacts.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 24s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |  40m 13s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  17m  9s |  |  the patch passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  javac  |  17m  9s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |  16m 15s |  |  the patch passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  javac  |  16m 15s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   4m 34s |  |  root: The patch generated 
0 new + 1638 unchanged - 90 fixed = 1638 total (was 1728)  |
   | +1 :green_heart: |  mvnsite  |  40m  5s |  |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   0m 40s |  

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-27 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2440329588

   I was busy for the past two weeks due to some work-related matters. This 
week, I will continue to improve this PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-09 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1793739270


##
hadoop-mapreduce-project/hadoop-mapreduce-client/pom.xml:
##
@@ -154,10 +154,21 @@
   provided
 
 
-  com.sun.jersey.jersey-test-framework
-  jersey-test-framework-grizzly2
+  org.glassfish.jersey.test-framework
+  jersey-test-framework-core
   test
 
+
+  org.glassfish.jersey.test-framework.providers
+  jersey-test-framework-provider-grizzly2
+  test
+  

Review Comment:
   You’re right. I will make the necessary changes to ensure the POM is concise.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-09 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1793735243


##
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/http/TestHttpServer.java:
##
@@ -448,17 +443,15 @@ public List getGroups(String user) throws 
IOException {
 
 @Override
 public Set getGroupsSet(String user) throws IOException {
-  Set result = new HashSet();
-  result.addAll(mapping.get(user));
-  return result;
+  return new HashSet<>(mapping.get(user));
 }
   }
 
   /**
* Verify the access for /logs, /stacks, /conf, and /logLevel
* servlets, when authentication filters are set, but authorization is not
* enabled.
-   * @throws Exception 
+   * @throws Exception if there is an error during, an exception will be 
thrown.

Review Comment:
   I added this comment because Yetus prompts us about JavaDoc compilation 
errors. While I included this description, I should focus more on the 
surrounding code context. I will improve it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-09 Thread via GitHub


slfan1989 commented on code in PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#discussion_r1793730438


##
hadoop-common-project/hadoop-common/pom.xml:
##
@@ -93,104 +93,69 @@
   compile
 
 
-  javax.servlet
-  javax.servlet-api
+  jakarta.servlet.jsp
+  jakarta.servlet.jsp-api
   compile
 
 
-  jakarta.activation
-  jakarta.activation-api
-  runtime
-
-
-  org.eclipse.jetty
-  jetty-server
+  jakarta.ws.rs
+  jakarta.ws.rs-api
   compile
 
 
-  org.eclipse.jetty
-  jetty-util
-  compile
-
-
-  org.eclipse.jetty
-  jetty-servlet
+  org.glassfish.jersey.core

Review Comment:
   Actually, I'm not sure if this is the most streamlined version. Sometimes, 
local compilation passes, but after submitting to Yetus, I still encounter 
"class not found" errors or issues with certain rules during unit tests. I 
added dependencies one by one based on the errors until they disappeared. I can 
try to make these dependencies a bit more streamlined.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-09 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2402636984

   > > the overall renovation time has taken longer than expected
   > 
   > I can't think of any big change where things took less than expected.
   > 
   > I'll have a quick look at it, but it's big enough I'll only be trying to 
spot the obvious issues i've hit before. after that it's well "merge and see"
   
   I initially thought I could complete this PR more quickly since the Yarn 
modules are similar, and modifying one would make the others relatively easier, 
until I encountered the timeline module(somewhat different from the previous 
ones). 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-10-09 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2402623880

   @steveloughran 
   
   Thank you very much for reviewing this PR! Based on the recent compilation 
results, we have largely resolved the unit test errors, sputbug, and 
compilation issues (the latest issue seems unrelated to our changes. I 
appreciate your suggestions! I'm preparing a new version that will incorporate 
your feedback, continue to streamline some code, update the JAR package 
version, and document these changes in LICENSE-binary. I am optimistic about 
the impact of these changes on code stability, as most improvements focus on 
unit tests with minimal modifications to business logic. I hope we can smoothly 
complete this PR and merge it into both trunk and branch-3.4.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-27 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2380427389

   I need your help to review this PR together. I know this PR contains many 
changes. What else can I do to help you review it more effectively? Could you 
give me some suggestions? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-27 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2380355237

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/22/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 31s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  41m 18s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  20m 37s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  compile  |  18m 50s |  |  trunk passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  checkstyle  |   5m 10s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  35m 39s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  29m  6s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | -1 :x: |  javadoc  |   0m 14s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/22/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05.  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   0m 49s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/22/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |  12m  2s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/22/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html)
 |  hadoop-yarn-project/hadoop-yarn in trunk has 1 extant spotbugs warnings.  |
   | -1 :x: |  spotbugs  |   1m  1s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/22/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-27 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2379596519

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 34s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  1s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/21/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 47s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 25s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 49s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  compile  |  16m 20s |  |  trunk passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  checkstyle  |   5m 15s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  36m 24s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  30m 11s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | -1 :x: |  javadoc  |   0m 17s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/21/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05.  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   0m 47s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/21/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |  11m 19s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/21/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html)
 |  hadoop-yarn-project/hadoop-yarn in trunk has 1 extant spotbugs warnings.  |
   | -1 :x: |  spotbugs  |   1m  1s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/21/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  
branch/

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-25 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2375081319

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 37s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  5s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  5s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  5s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  5s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/20/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  15m 57s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  39m 15s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  19m 40s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  compile  |  18m 41s |  |  trunk passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  checkstyle  |   5m 13s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  34m  5s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  28m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | -1 :x: |  javadoc  |   0m 14s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/20/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05.  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   0m 41s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/20/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |  11m 42s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/20/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html)
 |  hadoop-yarn-project/hadoop-yarn in trunk has 1 extant spotbugs warnings.  |
   | -1 :x: |  spotbugs  |   1m  0s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/20/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-check-test-invariants no spotbugs 
output file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-25 Thread via GitHub


hadoop-yetus commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2374817455

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 33s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  6s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  6s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  6s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  6s |  |  Shelldocs was not available.  |
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +0 :ok: |  jsonlint  |   0m  0s |  |  jsonlint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | -1 :x: |  pathlen  |   0m  0s | 
[/results-pathlen.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/19/artifact/out/results-pathlen.txt)
 |  The patch appears to contain 1 files with names longer than 240  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 115 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  16m  4s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 42s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 33s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | +1 :green_heart: |  compile  |  16m 44s |  |  trunk passed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05  |
   | +1 :green_heart: |  checkstyle  |   4m 43s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  33m 56s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  29m  2s |  |  trunk passed with JDK 
Ubuntu-11.0.24+8-post-Ubuntu-1ubuntu320.04  |
   | -1 :x: |  javadoc  |   0m 15s | 
[/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/19/artifact/out/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase_hadoop-yarn-server-timelineservice-hbase-server_hadoop-yarn-server-timelineservice-hbase-server-2-jdkPrivateBuild-1.8.0_422-8u422-b05-1~20.04-b05.txt)
 |  hadoop-yarn-server-timelineservice-hbase-server-2 in trunk failed with JDK 
Private Build-1.8.0_422-8u422-b05-1~20.04-b05.  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   0m 47s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/19/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |  12m 23s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/19/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html)
 |  hadoop-yarn-project/hadoop-yarn in trunk has 1 extant spotbugs warnings.  |
   | -1 :x: |  spotbugs  |   1m  0s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7019/19/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 24s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 25s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  39m 31s |  |  bra

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-09-24 Thread via GitHub


slfan1989 commented on PR #7019:
URL: https://github.com/apache/hadoop/pull/7019#issuecomment-2370738063

   @ayushtkn @aajisaka @virajjasani @steveloughran 
   
   I think we can start reviewing this PR now. Most issues have been resolved, 
but there are indeed some problems that I will continue to address. However, I 
would like to hear your thoughts on this pr.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-08-12 Thread via GitHub


slfan1989 commented on PR #6989:
URL: https://github.com/apache/hadoop/pull/6989#issuecomment-2283597443

   @ayushtkn Thank you for your feedback! I will continue to improve this PR. 
It might be because my current computer has an M3 chip, so it compiles 
successfully. I will try compiling with an x86 architecture.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-08-12 Thread via GitHub


ayushtkn commented on PR #6989:
URL: https://github.com/apache/hadoop/pull/6989#issuecomment-2283585127

   Some issue with the yetus I belive, need to investigate.
   But It fails for me locally during compilation
   ```
   [INFO] --- enforcer:3.0.0:enforce (depcheck) @ hadoop-yarn-common ---
   [WARNING] 
   Dependency convergence error for 
javax.servlet:javax.servlet-api:jar:3.1.0:provided paths to dependency are:
   +-org.apache.hadoop:hadoop-yarn-common:jar:3.5.0-SNAPSHOT
 +-org.apache.hadoop:hadoop-common:jar:3.5.0-SNAPSHOT:provided
   +-org.eclipse.jetty:jetty-server:jar:9.4.53.v20231009:provided
 +-javax.servlet:javax.servlet-api:jar:3.1.0:provided
   and
   +-org.apache.hadoop:hadoop-yarn-common:jar:3.5.0-SNAPSHOT
 +-org.apache.hadoop:hadoop-common:jar:3.5.0-SNAPSHOT:provided
   +-org.eclipse.jetty:jetty-server:jar:9.4.53.v20231009:provided
 +-org.eclipse.jetty:jetty-http:jar:9.4.53.v20231009:provided
   +-javax.servlet:javax.servlet-api:jar:3.1.0:provided
   and
   +-org.apache.hadoop:hadoop-yarn-common:jar:3.5.0-SNAPSHOT
 +-org.apache.hadoop:hadoop-common:jar:3.5.0-SNAPSHOT:provided
   +-org.eclipse.jetty:jetty-servlet:jar:9.4.53.v20231009:provided
 +-org.eclipse.jetty:jetty-util-ajax:jar:9.4.53.v20231009:provided
   +-javax.servlet:javax.servlet-api:jar:3.1.0:provided
   and
   +-org.apache.hadoop:hadoop-yarn-common:jar:3.5.0-SNAPSHOT
 +-org.eclipse.jetty:jetty-util:jar:9.4.53.v20231009:compile
   +-javax.servlet:javax.servlet-api:jar:3.1.0:provided
   and
   +-org.apache.hadoop:hadoop-yarn-common:jar:3.5.0-SNAPSHOT
 
+-org.glassfish.jersey.test-framework.providers:jersey-test-framework-provider-grizzly2:jar:2.41:test
   
+-org.glassfish.jersey.containers:jersey-container-grizzly2-servlet:jar:2.41:test
 +-org.glassfish.grizzly:grizzly-http-servlet:jar:2.4.4:test
   +-javax.servlet:javax.servlet-api:jar:4.0.0-b07:test
   
   [WARNING] Rule 0: org.apache.maven.plugins.enforcer.DependencyConvergence 
failed with message:
   Failed while enforcing releasability. See above detailed error message.
   [INFO] --
   ''
   ''
   [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-enforcer-plugin:3.0.0:enforce (depcheck) on 
project hadoop-yarn-common: Some Enforcer rules have failed. Look above for 
specific messages explaining why the rule failed. -> [Help 1]
   [ERROR] 
   [ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
   [ERROR] Re-run Maven using the -X switch to enable full debug logging.
   [ERROR] 
   [ERROR] For more information about the errors and possible solutions, please 
read the following articles:
   [ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
   [ERROR] 
   [ERROR] After correcting the problems, you can resume the build with the 
command
   [ERROR]   mvn  -rf :hadoop-yarn-common
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-08-12 Thread via GitHub


hadoop-yetus commented on PR #6989:
URL: https://github.com/apache/hadoop/pull/6989#issuecomment-2283560846

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 21s |  |  
https://github.com/apache/hadoop/pull/6989 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-6989/8/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-08-12 Thread via GitHub


slfan1989 commented on PR #6989:
URL: https://github.com/apache/hadoop/pull/6989#issuecomment-2283524997

   @steveloughran @ayushtkn 
   
   I have good news: using Jersey 2.4.1, I can now compile it correctly on my 
local machine. I’ve modified many unit tests, which also pass locally. Of 
course, there are still some modules that need improvement and cannot be 
reviewed yet. However, I don't understand why Yetus isn't running. I've already 
rebased the code but it still doesn't work. 
   
   Could you help me get Yetus running? I want to see how many tests will fail 
so that I can continue improving this PR.
   
   cc: @Hexiaoqiao 
   
   ```
   [INFO] 

   [INFO] Reactor Summary for Apache Hadoop Main 3.5.0-SNAPSHOT:
   [INFO] 
   [INFO] Apache Hadoop Main . SUCCESS [  1.376 
s]
   [INFO] Apache Hadoop Build Tools .. SUCCESS [  1.439 
s]
   [INFO] Apache Hadoop Project POM .. SUCCESS [  0.753 
s]
   [INFO] Apache Hadoop Annotations .. SUCCESS [  1.404 
s]
   [INFO] Apache Hadoop Assemblies ... SUCCESS [  0.101 
s]
   [INFO] Apache Hadoop Project Dist POM . SUCCESS [  0.953 
s]
   [INFO] Apache Hadoop Maven Plugins  SUCCESS [  1.974 
s]
   [INFO] Apache Hadoop MiniKDC .. SUCCESS [  0.818 
s]
   [INFO] Apache Hadoop Auth . SUCCESS [  2.966 
s]
   [INFO] Apache Hadoop Auth Examples  SUCCESS [  0.838 
s]
   [INFO] Apache Hadoop Common ... SUCCESS [ 17.908 
s]
   [INFO] Apache Hadoop NFS .. SUCCESS [  2.779 
s]
   [INFO] Apache Hadoop KMS .. SUCCESS [  2.755 
s]
   [INFO] Apache Hadoop Registry . SUCCESS [  3.210 
s]
   [INFO] Apache Hadoop Common Project ... SUCCESS [  0.021 
s]
   [INFO] Apache Hadoop HDFS Client .. SUCCESS [ 13.285 
s]
   [INFO] Apache Hadoop HDFS . SUCCESS [ 35.347 
s]
   [INFO] Apache Hadoop HDFS Native Client ... SUCCESS [  1.877 
s]
   [INFO] Apache Hadoop HttpFS ... SUCCESS [  3.737 
s]
   [INFO] Apache Hadoop HDFS-NFS . SUCCESS [  2.470 
s]
   [INFO] Apache Hadoop YARN . SUCCESS [  0.028 
s]
   [INFO] Apache Hadoop YARN API . SUCCESS [  8.224 
s]
   [INFO] Apache Hadoop YARN Common .. SUCCESS [ 10.812 
s]
   [INFO] Apache Hadoop YARN Server .. SUCCESS [  0.030 
s]
   [INFO] Apache Hadoop YARN Server Common ... SUCCESS [  6.952 
s]
   [INFO] Apache Hadoop YARN ApplicationHistoryService ... SUCCESS [  4.823 
s]
   [INFO] Apache Hadoop YARN Timeline Service  SUCCESS [  3.086 
s]
   [INFO] Apache Hadoop YARN Web Proxy ... SUCCESS [  2.505 
s]
   [INFO] Apache Hadoop YARN ResourceManager . SUCCESS [ 13.206 
s]
   [INFO] Apache Hadoop YARN NodeManager . SUCCESS [  9.869 
s]
   [INFO] Apache Hadoop YARN Server Tests  SUCCESS [  2.642 
s]
   [INFO] Apache Hadoop YARN Client .. SUCCESS [  4.814 
s]
   [INFO] Apache Hadoop MapReduce Client . SUCCESS [  1.365 
s]
   [INFO] Apache Hadoop MapReduce Core ... SUCCESS [  4.232 
s]
   [INFO] Apache Hadoop MapReduce Common . SUCCESS [  4.849 
s]
   [INFO] Apache Hadoop MapReduce Shuffle  SUCCESS [  3.461 
s]
   [INFO] Apache Hadoop MapReduce App  SUCCESS [  4.904 
s]
   [INFO] Apache Hadoop MapReduce HistoryServer .. SUCCESS [  3.686 
s]
   [INFO] Apache Hadoop MapReduce JobClient .. SUCCESS [  4.167 
s]
   [INFO] Apache Hadoop Distributed Copy . SUCCESS [  3.399 
s]
   [INFO] Apache Hadoop Mini-Cluster . SUCCESS [  2.160 
s]
   [INFO] Apache Hadoop Federation Balance ... SUCCESS [  2.962 
s]
   [INFO] Apache Hadoop HDFS-RBF . SUCCESS [  7.082 
s]
   [INFO] Apache Hadoop HDFS Project . SUCCESS [  0.021 
s]
   [INFO] Apache Hadoop YARN SharedCacheManager .. SUCCESS [  2.506 
s]
   [INFO] Apache Hadoop YARN Timeline Plugin Storage . SUCCESS [  2.443 
s]
   [INFO] Apache Hadoop YARN TimelineService HBase Backend ... SUCCESS [  0.026 
s]
   [INFO] Apache Hadoop YARN TimelineService HBase Common  SUCCESS [  3.060 
s]
   [INFO] Apache Hadoop YARN TimelineService HBase Client  SUCCESS [  3.579 
s]
   [INFO] Apache Hadoop YARN TimelineService HBase Servers ... SUCC

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]

2024-08-12 Thread via GitHub


hadoop-yetus commented on PR #6989:
URL: https://github.com/apache/hadoop/pull/6989#issuecomment-2283481882

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 21s |  |  
https://github.com/apache/hadoop/pull/6989 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-6989/7/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-21 Thread via GitHub


slfan1989 closed pull request #6606: HADOOP-15984. Update jersey from 1.19 to 
2.x
URL: https://github.com/apache/hadoop/pull/6606


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-09 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1987044897

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  18m 45s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  3s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 80 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m  7s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  36m 39s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  19m 38s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  17m 50s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   4m 52s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  32m 19s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  29m 12s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  27m 22s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 29s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/9/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 57s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/9/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 31s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/9/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 12s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/9/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 39s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  38m 19s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  38m 48s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 24s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 41s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/j

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-09 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1987036175

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 32s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  3s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 80 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 17s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m 57s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 28s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  16m  9s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   4m 34s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  34m  4s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  30m 33s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  29m 11s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 32s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/10/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m  0s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/10/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 33s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/10/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 13s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/10/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 43s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 40s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  33m  1s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  33m 27s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 29s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 42s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibran

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-09 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1986996454

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 22s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  2s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  1s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  1s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 80 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  13m 46s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  19m 37s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |   8m 56s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |   8m 20s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   2m 36s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  20m 16s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  19m 16s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  19m 21s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   1m 25s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/11/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 38s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/11/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 59s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/11/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 45s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/11/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 26s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 29s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 27s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 28s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  19m 45s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  20m  1s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 16s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 24s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibran

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-09 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1986962989

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 55s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  3s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  1s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  1s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 80 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 29s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  31m 34s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 25s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  17m 20s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   4m 32s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  33m 56s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  30m 28s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  28m 58s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 42s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 26s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/8/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m  0s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/8/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 32s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/8/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 14s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/8/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 41s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  33m 13s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  33m 41s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 27s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 42s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/j

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-07 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1984948590

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 35s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  4s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 78 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 50s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  34m 24s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 46s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  16m 57s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   4m 43s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  30m 52s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  27m 38s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  25m 27s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 30s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/7/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 54s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/7/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 30s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/7/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 10s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/7/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 36s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  35m  3s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  35m 28s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 27s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 40s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/j

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-07 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1983367997

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 33s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  3s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  1s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  1s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  1s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 75 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 57s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  37m 30s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  18m 44s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  16m 53s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   5m 28s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  31m  1s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  26m 31s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  24m 47s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 37s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 40s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/5/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 55s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/5/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 32s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/5/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m  7s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/5/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 33s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  36m 54s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  37m 21s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 27s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 41s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/j

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-06 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1982178875

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  12m 46s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  3s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  xmllint  |   0m  0s |  |  xmllint was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 63 new or modified test files.  |
    _ trunk Compile Tests _ |
   | +0 :ok: |  mvndep  |  14m 54s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  33m 49s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |  17m 46s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  compile  |  16m 21s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   5m  1s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |  29m 35s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |  26m 46s |  |  trunk passed with JDK 
Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1  |
   | +1 :green_heart: |  javadoc  |  24m 55s |  |  trunk passed with JDK 
Private Build-1.8.0_392-8u392-ga-1~20.04-b08  |
   | +0 :ok: |  spotbugs  |   0m 38s |  |  branch/hadoop-project no spotbugs 
output file (spotbugsXml.xml)  |
   | -1 :x: |  spotbugs  |   2m 34s | 
[/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/4/artifact/out/branch-spotbugs-hadoop-common-project_hadoop-common-warnings.html)
 |  hadoop-common-project/hadoop-common in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   0m 54s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/4/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-httpfs-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-httpfs in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 30s | 
[/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/4/artifact/out/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-rbf-warnings.html)
 |  hadoop-hdfs-project/hadoop-hdfs-rbf in trunk has 1 extant spotbugs 
warnings.  |
   | -1 :x: |  spotbugs  |   1m 11s | 
[/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/4/artifact/out/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-services_hadoop-yarn-services-core-warnings.html)
 |  
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core
 in trunk has 1 extant spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-api no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 32s |  |  
branch/hadoop-client-modules/hadoop-client-runtime no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-check-invariants no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 34s |  |  
branch/hadoop-client-modules/hadoop-client-minicluster no spotbugs output file 
(spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 36s |  |  
branch/hadoop-client-modules/hadoop-client-integration-tests no spotbugs output 
file (spotbugsXml.xml)  |
   | +0 :ok: |  spotbugs  |   0m 35s |  |  
branch/hadoop-cloud-storage-project/hadoop-cloud-storage no spotbugs output 
file (spotbugsXml.xml)  |
   | +1 :green_heart: |  shadedclient  |  34m 19s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  34m 44s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   1m 29s |  |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 40s | 
[/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/j

Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-06 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1980575783

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 23s |  |  
https://github.com/apache/hadoop/pull/6606 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 |
   |--:|:-|
   | GITHUB PR | https://github.com/apache/hadoop/pull/6606 |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/3/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-03 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1975123133

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 22s |  |  
https://github.com/apache/hadoop/pull/6606 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 |
   |--:|:-|
   | GITHUB PR | https://github.com/apache/hadoop/pull/6606 |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/2/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-03 Thread via GitHub


hadoop-yetus commented on PR #6606:
URL: https://github.com/apache/hadoop/pull/6606#issuecomment-1975084415

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m  0s |  |  Docker mode activated.  |
   | -1 :x: |  patch  |   0m 21s |  |  
https://github.com/apache/hadoop/pull/6606 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 |
   |--:|:-|
   | GITHUB PR | https://github.com/apache/hadoop/pull/6606 |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6606/1/console |
   | versions | git=2.34.1 |
   | Powered by | Apache Yetus 0.14.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



[PR] HADOOP-15984. Update jersey from 1.19 to 2.x [hadoop]

2024-03-03 Thread via GitHub


slfan1989 opened a new pull request, #6606:
URL: https://github.com/apache/hadoop/pull/6606

   
   
   ### Description of PR
   JIRA: HADOOP-15984. Update jersey from 1.19 to 2.x
   
   ### 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 under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org