Re: [PR] HADOOP-15984. Update jersey from 1.19 to 2.x. [hadoop]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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]
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