Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-13 Thread via GitHub


abstractdog merged PR #407:
URL: https://github.com/apache/tez/pull/407


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-13 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2876166090

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 11s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 3 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 30s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   7m 47s |  |  master passed  |
   | +1 :green_heart: |  compile  |   2m 59s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m  5s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   2m 21s |  |  master passed  |
   | -1 :x: |  spotbugs  |   0m 53s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 24s | 
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-tez-runtime-library.txt)
 |  tez-runtime-library in master failed.  |
   | -1 :x: |  spotbugs  |   0m 25s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 21s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 23s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 17s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m 33s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 28s |  |  The patch generated 0 new 
+ 7 unchanged - 1 fixed = 7 total (was 8)  |
   | +1 :green_heart: |  compile  |   3m  6s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   3m  6s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   0m 16s |  |  The patch passed 
checkstyle in tez-api  |
   | +1 :green_heart: |  checkstyle  |   0m 16s |  |  tez-runtime-library: The 
patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)  |
   | +1 :green_heart: |  checkstyle  |   0m 17s |  |  The patch passed 
checkstyle in tez-dag  |
   | +1 :green_heart: |  checkstyle  |   0m 12s |  |  The patch passed 
checkstyle in tez-tests  |
   | +1 :green_heart: |  checkstyle  |   0m 13s |  |  The patch passed 
checkstyle in tez-ext-service-tests  |
   | +1 :green_heart: |  checkstyle  |   0m 28s |  |  root: The patch generated 
0 new + 68 unchanged - 1 fixed = 68 total (was 69)  |
   | +1 :green_heart: |  hadolint  |   0m  1s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   2m 16s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |   8m 33s |  |  the patch passed  |
    _ Other Tests _ |
   | +1 :green_heart: |  unit  |   1m 59s |  |  tez-api in the patch passed.  |
   | +1 :green_heart: |  unit  |   4m 18s |  |  tez-runtime-library in the 
patch passed.  |
   | +1 :green_heart: |  unit  |   3m 57s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  33m 27s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   3m 45s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |  58m 13s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/patch-unit-root.txt)
 |  root in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   1m 51s |  |  The patch does not 
generate ASF License warnings.  

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-13 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2875661085

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-13 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2875503274

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |  12m 12s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  1s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 3 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 42s |  |  Maven dependency ordering for branch  |
   | -1 :x: |  mvninstall  |   6m 33s | 
[/branch-mvninstall-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-mvninstall-root.txt)
 |  root in master failed.  |
   | +1 :green_heart: |  compile  |   3m  6s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   1m 59s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   2m 28s |  |  master passed  |
   | -1 :x: |  spotbugs  |   0m 54s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 26s | 
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-tez-runtime-library.txt)
 |  tez-runtime-library in master failed.  |
   | -1 :x: |  spotbugs  |   0m 28s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 22s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 22s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 18s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 12s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m 14s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 25s |  |  The patch generated 0 new 
+ 7 unchanged - 1 fixed = 7 total (was 8)  |
   | +1 :green_heart: |  compile  |   3m  0s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   3m  0s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  1s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   0m 15s |  |  The patch passed 
checkstyle in tez-api  |
   | +1 :green_heart: |  checkstyle  |   0m 17s |  |  tez-runtime-library: The 
patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  checkstyle  |   0m 16s |  |  The patch passed 
checkstyle in tez-dag  |
   | +1 :green_heart: |  checkstyle  |   0m 12s |  |  The patch passed 
checkstyle in tez-tests  |
   | +1 :green_heart: |  checkstyle  |   0m 13s |  |  The patch passed 
checkstyle in tez-ext-service-tests  |
   | +1 :green_heart: |  checkstyle  |   0m 28s |  |  root: The patch generated 
0 new + 67 unchanged - 1 fixed = 67 total (was 68)  |
   | +1 :green_heart: |  hadolint  |   0m  2s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   2m 12s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |   8m 40s |  |  the patch passed  |
    _ Other Tests _ |
   | +1 :green_heart: |  unit  |   2m  0s |  |  tez-api in the patch passed.  |
   | +1 :green_heart: |  unit  |   4m 30s |  |  tez-runtime-library in the 
patch passed.  |
   | +1 :green_heart: |  unit  |   4m  1s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  32m 56s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   3m 48s |  |  tez-ext-service-tests in the 
patch passed.  |
   | +1 :green_heart: |  unit  |  59m 12s |  |  root in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   1m 52s |  |  The patch does not 
generate 

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-12 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2875103530

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-12 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2874794678

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/30/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-12 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2873791038

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 42s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  1s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 3 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 47s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 39s |  |  master passed  |
   | +1 :green_heart: |  compile  |   5m 21s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   3m 31s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   3m 48s |  |  master passed  |
   | -1 :x: |  spotbugs  |   1m 11s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 42s | 
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-tez-runtime-library.txt)
 |  tez-runtime-library in master failed.  |
   | -1 :x: |  spotbugs  |   0m 49s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 39s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 38s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 32s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   6m  8s |  |  the patch passed  |
   | -1 :x: |  codespell  |   0m 54s | 
[/results-codespell.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/results-codespell.txt)
 |  The patch generated 1 new + 8 unchanged - 0 fixed = 9 total (was 8)  |
   | +1 :green_heart: |  compile  |   5m 20s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   5m 20s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   0m 29s | 
[/results-checkstyle-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/results-checkstyle-tez-runtime-library.txt)
 |  tez-runtime-library: The patch generated 1 new + 1 unchanged - 0 fixed = 2 
total (was 1)  |
   | -0 :warning: |  checkstyle  |   0m 53s | 
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/results-checkstyle-root.txt)
 |  root: The patch generated 1 new + 69 unchanged - 0 fixed = 70 total (was 
69)  |
   | +1 :green_heart: |  hadolint  |   0m  1s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   3m 40s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |  15m  0s |  |  the patch passed  |
    _ Other Tests _ |
   | +1 :green_heart: |  unit  |   2m 30s |  |  tez-api in the patch passed.  |
   | +1 :green_heart: |  unit  |   6m  9s |  |  tez-runtime-library in the 
patch passed.  |
   | +1 :green_heart: |  unit  |   5m 14s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  40m 27s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   4m 38s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |  72m 31s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/artifact/out/patch-unit-root.txt)
 |  root in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   2m 47s |  |  The patch does not 
generate ASF License warni

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-12 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2873211422

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-10 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2869139050

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 40s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 3 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 44s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 42s |  |  master passed  |
   | +1 :green_heart: |  compile  |   5m 19s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   3m 31s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   3m 46s |  |  master passed  |
   | -1 :x: |  spotbugs  |   1m 11s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 43s | 
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-tez-runtime-library.txt)
 |  tez-runtime-library in master failed.  |
   | -1 :x: |  spotbugs  |   0m 49s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 40s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 38s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 30s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   6m 14s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 54s |  |  No new issues.  |
   | +1 :green_heart: |  compile  |   5m 17s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   5m 17s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   3m 13s |  |  the patch passed  |
   | +1 :green_heart: |  hadolint  |   0m  0s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   3m 40s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |  15m  3s |  |  the patch passed  |
    _ Other Tests _ |
   | +1 :green_heart: |  unit  |   2m 31s |  |  tez-api in the patch passed.  |
   | +1 :green_heart: |  unit  |   6m  7s |  |  tez-runtime-library in the 
patch passed.  |
   | +1 :green_heart: |  unit  |   5m 14s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  40m 40s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   4m 29s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |  74m 26s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/patch-unit-root.txt)
 |  root in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   2m 48s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 204m 45s |  |  |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | tez.analyzer.TestAnalyzer |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.49 ServerAPI=1.49 base: 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/tez/pull/407 |
   | Optional Tests | dupname asflicense codespell detsecrets hadolint 
shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs 
checkstyle |
   | uname | Linux 1c6d2ce97a8f 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 
15:53:30 UTC 2025 x

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-10 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2869016236

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-09 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2866368547

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/27/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2863827190

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 39s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 2 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 15s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 44s |  |  master passed  |
   | +1 :green_heart: |  compile  |   4m 40s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m 58s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   3m 13s |  |  master passed  |
   | -1 :x: |  spotbugs  |   1m 10s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 31s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
   | -1 :x: |  spotbugs  |   0m 48s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 37s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 39s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   5m 47s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 54s |  |  No new issues.  |
   | +1 :green_heart: |  compile  |   4m 42s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   4m 42s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   2m 44s |  |  the patch passed  |
   | +1 :green_heart: |  hadolint  |   0m  1s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   3m  5s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |  13m 40s |  |  the patch passed  |
    _ Other Tests _ |
   | +1 :green_heart: |  unit  |   2m 29s |  |  tez-api in the patch passed.  |
   | -1 :x: |  unit  |   5m 42s | 
[/patch-unit-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/patch-unit-tez-dag.txt)
 |  tez-dag in the patch failed.  |
   | +1 :green_heart: |  unit  |  41m 28s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   4m 29s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |  10m 21s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/patch-unit-root.txt)
 |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   2m 19s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 128m 35s |  |  |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | tez.dag.history.recovery.TestRecoveryService |
   |   | tez.dag.history.recovery.TestRecoveryService |
   |   | 
tez.runtime.library.common.shuffle.impl.TestSimpleFetchedInputAllocator |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.49 ServerAPI=1.49 base: 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/tez/pull/407 |
   | Optional Tests | dupname asflicense codespell detsecrets hadolint 
shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs 
checkstyle |
   | uname | Linux 8212d95e822d 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 
15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2863513939

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862956716

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 42s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  0s |  |  Shelldocs 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 2 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 17s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 45s |  |  master passed  |
   | +1 :green_heart: |  compile  |   4m 38s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   3m  1s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   3m 13s |  |  master passed  |
   | -1 :x: |  spotbugs  |   1m 11s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 30s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
   | -1 :x: |  spotbugs  |   0m 49s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 38s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 39s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   5m 52s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 54s |  |  No new issues.  |
   | +1 :green_heart: |  compile  |   4m 41s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   4m 41s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   2m 42s |  |  the patch passed  |
   | +1 :green_heart: |  hadolint  |   0m  2s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   3m  4s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |  13m 36s |  |  the patch passed  |
    _ Other Tests _ |
   | -1 :x: |  unit  |   2m 28s | 
[/patch-unit-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/patch-unit-tez-api.txt)
 |  tez-api in the patch failed.  |
   | +1 :green_heart: |  unit  |   5m 14s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  38m 59s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   4m 31s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |   3m 45s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/patch-unit-root.txt)
 |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   2m 20s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 119m 15s |  |  |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | tez.client.TestTezClientUtils |
   |   | tez.client.TestTezClientUtils |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.49 ServerAPI=1.49 base: 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/tez/pull/407 |
   | Optional Tests | dupname asflicense codespell detsecrets hadolint 
shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs 
checkstyle |
   | uname | Linux 900fe03ab61b 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 
15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-407/src/.yetus/personality.sh
 |
   | git

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862626189

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


maheshrajus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862649430

   @ayushtkn @abstractdog I've addressed all your comments and pushed the 
latest commit. Can you please check and approve the PR. Thank you !


-- 
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: issues-unsubscr...@tez.apache.org

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



Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862419503

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   0m 11s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 2 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m  9s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   8m 19s |  |  master passed  |
   | +1 :green_heart: |  compile  |   3m  9s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   1m 39s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   2m  6s |  |  master passed  |
   | -1 :x: |  spotbugs  |   0m 57s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 20s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
   | -1 :x: |  spotbugs  |   0m 32s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 24s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 26s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 13s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m  2s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 39s |  |  No new issues.  |
   | +1 :green_heart: |  compile  |   3m 21s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   3m 21s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   1m 39s |  |  the patch passed  |
   | +1 :green_heart: |  hadolint  |   0m  0s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  2s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   1m 42s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |   8m 46s |  |  the patch passed  |
    _ Other Tests _ |
   | -1 :x: |  unit  |   0m 29s | 
[/patch-unit-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/patch-unit-tez-api.txt)
 |  tez-api in the patch failed.  |
   | +1 :green_heart: |  unit  |   4m  2s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  31m 59s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   3m 41s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |   1m 23s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/patch-unit-root.txt)
 |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   1m 13s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   |  85m 50s |  |  |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.49 ServerAPI=1.49 base: 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/tez/pull/407 |
   | Optional Tests | dupname asflicense codespell detsecrets hadolint 
shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs 
checkstyle |
   | uname | Linux eb5747f323e5 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 
15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-407/src/.yetus/personality.sh
 |
   | git revision | master / f54dee65e789fb5025a178afcdbb6514ccf2a57a |
   | Default Java | Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04 |
   |  Test Results | 
https

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862378542

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |::|--:|:|::|:---:|
   | +0 :ok: |  reexec  |   4m 36s |  |  Docker mode activated.  |
    _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +0 :ok: |  shelldocs  |   0m  1s |  |  Shelldocs 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 2 new or modified test files.  |
    _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 48s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 34s |  |  master passed  |
   | +1 :green_heart: |  compile  |   4m 43s |  |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m 59s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   3m 16s |  |  master passed  |
   | -1 :x: |  spotbugs  |   1m 11s | 
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/branch-spotbugs-tez-api.txt)
 |  tez-api in master failed.  |
   | -1 :x: |  spotbugs  |   0m 48s | 
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/branch-spotbugs-tez-dag.txt)
 |  tez-dag in master failed.  |
   | -1 :x: |  spotbugs  |   0m 38s | 
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/branch-spotbugs-tez-tests.txt)
 |  tez-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 37s | 
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
 |  tez-ext-service-tests in master failed.  |
   | -1 :x: |  spotbugs  |   0m 31s | 
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/branch-spotbugs-root.txt)
 |  root in master failed.  |
    _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   5m 51s |  |  the patch passed  |
   | +1 :green_heart: |  codespell  |   0m 54s |  |  No new issues.  |
   | +1 :green_heart: |  compile  |   4m 45s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   4m 45s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   2m 45s |  |  the patch passed  |
   | +1 :green_heart: |  hadolint  |   0m  2s |  |  The patch generated 0 new + 
0 unchanged - 1 fixed = 0 total (was 1)  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  |  No new issues.  |
   | +1 :green_heart: |  yamllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   3m  5s |  |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |  13m 43s |  |  the patch passed  |
    _ Other Tests _ |
   | -1 :x: |  unit  |   0m 45s | 
[/patch-unit-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/patch-unit-tez-api.txt)
 |  tez-api in the patch failed.  |
   | +1 :green_heart: |  unit  |   5m 13s |  |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  42m 52s |  |  tez-tests in the patch passed.  
|
   | +1 :green_heart: |  unit  |   4m 37s |  |  tez-ext-service-tests in the 
patch passed.  |
   | -1 :x: |  unit  |   2m  1s | 
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/patch-unit-root.txt)
 |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   2m 20s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 124m 15s |  |  |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.49 ServerAPI=1.49 base: 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/22/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/tez/pull/407 |
   | Optional Tests | dupname asflicense codespell detsecrets hadolint 
shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs 
checkstyle |
   | uname | Linux 90bd9439e7b7 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 
15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-407/src/.yetus/personality.sh
 |
   | git revision | master / ecb6a421db5012b8a07a55fdeaf924c7804361b5 |
   | Default Java | Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04 |
   |  Test Results | 
https

Re: [PR] TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime) [tez]

2025-05-08 Thread via GitHub


tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2862172369

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/23/console in case 
of problems.
   


-- 
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: issues-unsubscr...@tez.apache.org

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