Re: [PR] Tez: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079133317
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,17 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
long jvmMax = Runtime.getRuntime().maxMemory();
+long availableMemory;
+if ((jvmMax * 0.5d) >= Integer.MAX_VALUE) {
Review Comment:
maxMemory() is behaving differently with jdk17 compare to old jdk8.
The number is: 8589934592 bigger than Integer.MAX_VALUE(2147483647) which is
causing Number format exceptions while converting long to int.
I will add comment to the code lines. 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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2862045381 (!) 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/22/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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079099649
##
pom.xml:
##
@@ -101,6 +101,10 @@
1.1.10.4
${project.build.directory}/tmp
1.7.9
+
+ --add-opens=java.base/java.lang=ALL-UNNAMED
+ --add-opens=java.base/java.util=ALL-UNNAMED
+ --add-opens java.base/java.io=ALL-UNNAMED
Review Comment:
test.jvm.args is okay, and it's done, 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: issues-unsubscr...@tez.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Tez: Migrating to JDK17 [tez]
abstractdog commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2862113532 @maheshrajus : just 1 more question + please make sure that the final commit and github PR also contains a particular JIRA reference -- 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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079107278
##
tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/impl/TestSimpleFetchedInputAllocator.java:
##
@@ -66,7 +66,7 @@ public void testInMemAllocation() throws IOException {
FetchedInput fi2 = inputManager.allocate(requestSize, compressedSize, new
InputAttemptIdentifier(2, 1));
-assertEquals(FetchedInput.Type.MEMORY, fi2.getType());
+assertEquals(FetchedInput.Type.DISK, fi2.getType());
Review Comment:
I cannot see this change anymore, resolving this
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407: URL: https://github.com/apache/tez/pull/407#discussion_r2079108712 ## pom.xml: ## @@ -926,7 +930,9 @@ 1 false 900 --Xmx1024m -XX:+HeapDumpOnOutOfMemoryError +-Xmx4096m -XX:+HeapDumpOnOutOfMemoryError Review Comment: I cannot see this anymore, I guess it's working with 1024, resolving this -- 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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079107278
##
tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/impl/TestSimpleFetchedInputAllocator.java:
##
@@ -66,7 +66,7 @@ public void testInMemAllocation() throws IOException {
FetchedInput fi2 = inputManager.allocate(requestSize, compressedSize, new
InputAttemptIdentifier(2, 1));
-assertEquals(FetchedInput.Type.MEMORY, fi2.getType());
+assertEquals(FetchedInput.Type.DISK, fi2.getType());
Review Comment:
I cannot see this anymore, I guess it's working with 1024, resolving this
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079105663
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,17 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
long jvmMax = Runtime.getRuntime().maxMemory();
+long availableMemory;
+if ((jvmMax * 0.5d) >= Integer.MAX_VALUE) {
Review Comment:
how big is this number? is maxMemory() different in JDK17?
please also put a code comment here, without it, it's just a hack without
explanation :D
--
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: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2079056463
##
tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java:
##
@@ -396,8 +396,18 @@ public TezConfiguration(boolean loadDefaults) {
@ConfigurationProperty
public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS =
TEZ_AM_PREFIX + "launch.cluster-default.cmd-opts";
- public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
- "-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN";
+ public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT;
+
+ static {
+if (TezCommonUtils.getJavaVersion() >= 9) {
+ TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
+"-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN " +
+"--add-opens java.base/java.lang=ALL-UNNAMED";
+} else {
+ TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
+"-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN";
+}
+ }
Review Comment:
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: issues-unsubscr...@tez.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Tez: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2078912825
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,12 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
Review Comment:
Done
--
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: Migrating to JDK17 [tez]
ayushtkn commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2078900644
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,12 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
Review Comment:
nit.
avoid this
##
tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java:
##
@@ -396,8 +396,18 @@ public TezConfiguration(boolean loadDefaults) {
@ConfigurationProperty
public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS =
TEZ_AM_PREFIX + "launch.cluster-default.cmd-opts";
- public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
- "-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN";
+ public static final String TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT;
+
+ static {
+if (TezCommonUtils.getJavaVersion() >= 9) {
+ TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
+"-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN " +
+"--add-opens java.base/java.lang=ALL-UNNAMED";
+} else {
+ TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT =
+"-server -Djava.net.preferIPv4Stack=true
-Dhadoop.metrics.log.level=WARN";
+}
+ }
Review Comment:
This won't work in case someone configures the value of
`TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS`, in that case default won't kick in
itslef.
Should do the change in `TezClientUtils` in `constructAMLaunchOpts`
--
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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2859903634
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 2m 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 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 28s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 27s | | master passed |
| +1 :green_heart: | compile | 2m 44s | | master passed |
| +1 :green_heart: | checkstyle | 1m 50s | | master passed |
| +1 :green_heart: | javadoc | 2m 1s | | master passed |
| -1 :x: | spotbugs | 0m 53s |
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/21/artifact/out/branch-spotbugs-tez-api.txt)
| tez-api in master failed. |
| -1 :x: | spotbugs | 0m 29s |
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/21/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/21/artifact/out/branch-spotbugs-tez-tests.txt)
| tez-tests 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/21/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/21/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 19s | | the patch passed |
| +1 :green_heart: | codespell | 0m 27s | | No new issues. |
| +1 :green_heart: | compile | 2m 37s | | the patch passed |
| +1 :green_heart: | javac | 2m 37s | | the patch passed |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| +1 :green_heart: | checkstyle | 1m 21s | | 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 | 1m 53s | | the patch passed |
| +1 :green_heart: | spotbugs | 7m 45s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 30s | | tez-api in the patch passed. |
| +1 :green_heart: | unit | 4m 5s | | tez-dag in the patch passed. |
| +1 :green_heart: | unit | 31m 16s | | tez-tests in the patch passed.
|
| +1 :green_heart: | unit | 3m 45s | | tez-ext-service-tests in the
patch passed. |
| +1 :green_heart: | unit | 43m 3s | | root in the patch passed. |
| +1 :green_heart: | asflicense | 1m 29s | | The patch does not
generate ASF License warnings. |
| | | 125m 29s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.49 ServerAPI=1.49 base:
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/21/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 ce62427823e4 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 / 905c5aef2bf17c188d67059167bd8af936c62fb5 |
| Default Java | Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04 |
| Test Results |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/21/testReport/ |
| Max. process+thread count | 1169 (vs. ulimit of 5500) |
| modules | C: tez-api tez-dag tez-tests tez-ext-service-tests . U: . |
| Console output
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2859306195 (!) 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/21/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2857660216
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 24m 25s | | 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. |
| +0 :ok: | spotbugs | 0m 0s | | spotbugs executables are 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 51s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 9m 32s | | master passed |
| +1 :green_heart: | compile | 4m 42s | | master passed |
| +1 :green_heart: | checkstyle | 3m 1s | | master passed |
| +1 :green_heart: | javadoc | 3m 12s | | master passed |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 13s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 54s | | the patch passed |
| +1 :green_heart: | codespell | 0m 54s | | No new issues. |
| +1 :green_heart: | compile | 4m 44s | | the patch passed |
| +1 :green_heart: | javac | 4m 44s | | 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 0s | | No new issues. |
| +1 :green_heart: | javadoc | 3m 6s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 45s | | tez-api in the patch passed. |
| +1 :green_heart: | unit | 5m 11s | | tez-dag in the patch passed. |
| +1 :green_heart: | unit | 40m 31s | | tez-tests in the patch passed.
|
| +1 :green_heart: | unit | 4m 26s | | tez-ext-service-tests in the
patch passed. |
| +1 :green_heart: | unit | 52m 8s | | root in the patch passed. |
| +1 :green_heart: | asflicense | 2m 12s | | The patch does not
generate ASF License warnings. |
| | | 173m 11s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.49 ServerAPI=1.49 base:
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/20/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 fc07698b5f95 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 / ba9422adbd3dabfa51033c402a11069f87cf7f29 |
| Default Java | Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04 |
| Test Results |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/20/testReport/ |
| Max. process+thread count | 1207 (vs. ulimit of 5500) |
| modules | C: tez-api tez-dag tez-tests tez-ext-service-tests . U: . |
| Console output |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/20/console |
| versions | git=2.34.1 maven=3.6.3 hadolint=1.18.0-0-g76eee5c
codespell=2.0.0 shellcheck=0.7.1 yamllint=1.24.2 |
| Powered by | Apache Yetus 0.15.1 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: issues-unsubscr...@tez.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2857122362 (!) 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/20/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2856718167
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 15m 21s | | 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. |
| +0 :ok: | spotbugs | 0m 0s | | spotbugs executables are 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 26s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 31s | | master passed |
| +1 :green_heart: | compile | 2m 34s | | master passed |
| +1 :green_heart: | checkstyle | 1m 30s | | master passed |
| +1 :green_heart: | javadoc | 1m 43s | | master passed |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 8s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 32s | | the patch passed |
| +1 :green_heart: | codespell | 0m 35s | | No new issues. |
| +1 :green_heart: | compile | 3m 15s | | the patch passed |
| +1 :green_heart: | javac | 3m 15s | | the patch passed |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 15s |
[/results-checkstyle-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/artifact/out/results-checkstyle-tez-api.txt)
| tez-api: The patch generated 2 new + 35 unchanged - 0 fixed = 37 total (was
35) |
| -0 :warning: | checkstyle | 0m 28s |
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/artifact/out/results-checkstyle-root.txt)
| root: The patch generated 2 new + 36 unchanged - 0 fixed = 38 total (was
36) |
| +1 :green_heart: | hadolint | 0m 0s | | 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 | 1m 40s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 1m 57s | | tez-api in the patch passed. |
| -1 :x: | unit | 19m 16s |
[/patch-unit-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/artifact/out/patch-unit-tez-dag.txt)
| tez-dag in the patch passed. |
| -1 :x: | unit | 199m 17s |
[/patch-unit-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/artifact/out/patch-unit-tez-tests.txt)
| tez-tests in the patch passed. |
| +1 :green_heart: | unit | 30m 31s | | tez-ext-service-tests in the
patch passed. |
| -1 :x: | unit | 347m 4s |
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/artifact/out/patch-unit-root.txt)
| root in the patch passed. |
| +1 :green_heart: | asflicense | 1m 28s | | The patch does not
generate ASF License warnings. |
| | | 642m 56s | | |
| Reason | Tests |
|---:|:--|
| Failed junit tests | tez.dag.history.recovery.TestRecoveryService |
| | tez.test.TestExceptionPropagation |
| | tez.test.TestExceptionPropagation |
| |
tez.runtime.library.common.shuffle.impl.TestSimpleFetchedInputAllocator |
| | tez.dag.history.recovery.TestRecoveryService |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.49 ServerAPI=1.49 base:
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/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 fb290e776a21 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 / 47606b00b1f1ec06b49bf39da71c503913620a9f |
| Default Java | Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04 |
| Test Results |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/19/testReport/ |
| Max. process+thread count | 1128 (vs. uli
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2854741363 (!) 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/19/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: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2075559741
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,17 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
long jvmMax = Runtime.getRuntime().maxMemory();
+long availableMemory;
+if ((jvmMax * 0.5d) >= Integer.MAX_VALUE) {
Review Comment:
with JDK17 the available memory is coming big number from
(Runtime.getRuntime().maxMemory();)
So in our code we are converting the long value to integer and we are facing
numberFormat exceptions with jdk17. So i added check, >>long availableMemory =
(long) Math.min(jvmMax * 0.5d, Integer.MAX_VALUE);
Anyway this value we are using in creating MiniTezTestServiceCluster.
--
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: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407: URL: https://github.com/apache/tez/pull/407#discussion_r2075547039 ## build-tools/docker/Dockerfile: ## @@ -165,29 +165,18 @@ SHELL ["/bin/bash", "-o", "pipefail", "-c"] -# OpenJDK 8 - +# OpenJDK 17 # hadolint ignore=DL3008 -RUN apt-get -q update && apt-get -q install --no-install-recommends -y openjdk-8-jdk-headless \ -&& apt-get clean \ -&& rm -rf /var/lib/apt/lists/* +RUN apt-get update && \ +DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends -y openjdk-17-jdk && \ Review Comment: done. 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: issues-unsubscr...@tez.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Tez: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2075545519
##
pom.xml:
##
@@ -101,6 +101,10 @@
1.1.10.4
${project.build.directory}/tmp
1.7.9
+
+ --add-opens=java.base/java.lang=ALL-UNNAMED
+ --add-opens=java.base/java.util=ALL-UNNAMED
+ --add-opens java.base/java.io=ALL-UNNAMED
Review Comment:
can i put name as "test.jvm.args" ? any suggestions on name ? thx !
--
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: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407: URL: https://github.com/apache/tez/pull/407#discussion_r2075543394 ## pom.xml: ## @@ -926,7 +930,9 @@ 1 false 900 --Xmx1024m -XX:+HeapDumpOnOutOfMemoryError +-Xmx4096m -XX:+HeapDumpOnOutOfMemoryError Review Comment: This value i modified as compile and run with jdk17 it is taking more time for some of the tests and some tests are timeout. I will revert this value to "xmx1024m" and check once. if any no issue then i revert this value and modify the test case accordingly. -- 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: Migrating to JDK17 [tez]
maheshrajus commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2075542597
##
tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/impl/TestSimpleFetchedInputAllocator.java:
##
@@ -66,7 +66,7 @@ public void testInMemAllocation() throws IOException {
FetchedInput fi2 = inputManager.allocate(requestSize, compressedSize, new
InputAttemptIdentifier(2, 1));
-assertEquals(FetchedInput.Type.MEMORY, fi2.getType());
+assertEquals(FetchedInput.Type.DISK, fi2.getType());
Review Comment:
This test failed due to when i increased memory of surefire "-Xmx4096m
-XX:+HeapDumpOnOutOfMemoryError". Anyway this value i will revert and check
once. This value i modified as compile and run with jdk17 it is taking more
time for some of the tests and some tests are timeout. I will revert this value
to "xmx1024m" and check once. if any no issue then i revert this value and
modify the test case accordingly.
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407: URL: https://github.com/apache/tez/pull/407#discussion_r2068518423 ## build-tools/docker/Dockerfile: ## @@ -165,29 +165,18 @@ SHELL ["/bin/bash", "-o", "pipefail", "-c"] -# OpenJDK 8 - +# OpenJDK 17 # hadolint ignore=DL3008 -RUN apt-get -q update && apt-get -q install --no-install-recommends -y openjdk-8-jdk-headless \ -&& apt-get clean \ -&& rm -rf /var/lib/apt/lists/* +RUN apt-get update && \ +DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends -y openjdk-17-jdk && \ Review Comment: We might want to continue using `"&&"` at the beginning of lines. -- 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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2068515912
##
pom.xml:
##
@@ -101,6 +101,10 @@
1.1.10.4
${project.build.directory}/tmp
1.7.9
+
+ --add-opens=java.base/java.lang=ALL-UNNAMED
+ --add-opens=java.base/java.util=ALL-UNNAMED
+ --add-opens java.base/java.io=ALL-UNNAMED
Review Comment:
also, this looks to be specific to surefire testing processes, it would make
sense to reflect this in the property name, as jvm.args is too generic
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2068509203
##
pom.xml:
##
@@ -101,6 +101,10 @@
1.1.10.4
${project.build.directory}/tmp
1.7.9
+
+ --add-opens=java.base/java.lang=ALL-UNNAMED
+ --add-opens=java.base/java.util=ALL-UNNAMED
+ --add-opens java.base/java.io=ALL-UNNAMED
Review Comment:
nit: proper xml formatting
```
...
...
...
```
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407: URL: https://github.com/apache/tez/pull/407#discussion_r2068508063 ## pom.xml: ## @@ -926,7 +930,9 @@ 1 false 900 --Xmx1024m -XX:+HeapDumpOnOutOfMemoryError +-Xmx4096m -XX:+HeapDumpOnOutOfMemoryError Review Comment: Strange. Do we need 4G vs. 1G in the case of JDK17 for the surefire processes? Do we have an explanation for that? -- 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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2068499314
##
tez-ext-service-tests/src/test/java/org/apache/tez/tests/ExternalTezServiceTestHelper.java:
##
@@ -78,10 +78,17 @@ public ExternalTezServiceTestHelper(String testRootDir)
throws
for (Map.Entry entry : tezCluster.getConfig()) {
clusterConf.set(entry.getKey(), entry.getValue());
}
+
long jvmMax = Runtime.getRuntime().maxMemory();
+long availableMemory;
+if ((jvmMax * 0.5d) >= Integer.MAX_VALUE) {
Review Comment:
could this be simplified as:
```
long availableMemory = (long) Math.min(jvmMax * 0.5d, Integer.MAX_VALUE);
```
also, what is this for? how is it related to JDK17?
--
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: Migrating to JDK17 [tez]
abstractdog commented on code in PR #407:
URL: https://github.com/apache/tez/pull/407#discussion_r2068495409
##
tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/impl/TestSimpleFetchedInputAllocator.java:
##
@@ -66,7 +66,7 @@ public void testInMemAllocation() throws IOException {
FetchedInput fi2 = inputManager.allocate(requestSize, compressedSize, new
InputAttemptIdentifier(2, 1));
-assertEquals(FetchedInput.Type.MEMORY, fi2.getType());
+assertEquals(FetchedInput.Type.DISK, fi2.getType());
Review Comment:
hm, how is this related to the JDK17 upgrade?
--
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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2841487988
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 24m 58s | | 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. |
| +0 :ok: | spotbugs | 0m 0s | | spotbugs executables are 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 55s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 10m 9s | | master passed |
| +1 :green_heart: | compile | 5m 14s | | master passed |
| +1 :green_heart: | checkstyle | 3m 26s | | master passed |
| +1 :green_heart: | javadoc | 3m 46s | | master passed |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 6m 6s | | the patch passed |
| +1 :green_heart: | codespell | 0m 56s | | 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. |
| -0 :warning: | checkstyle | 0m 26s |
[/results-checkstyle-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/18/artifact/out/results-checkstyle-tez-api.txt)
| tez-api: The patch generated 2 new + 35 unchanged - 0 fixed = 37 total (was
35) |
| -0 :warning: | checkstyle | 0m 53s |
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/18/artifact/out/results-checkstyle-root.txt)
| root: The patch generated 2 new + 37 unchanged - 0 fixed = 39 total (was
37) |
| +1 :green_heart: | hadolint | 0m 1s | | 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 1s | | No new issues. |
| +1 :green_heart: | javadoc | 3m 38s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 2m 26s | | tez-api in the patch passed. |
| +1 :green_heart: | unit | 6m 8s | | tez-runtime-library in the
patch passed. |
| +1 :green_heart: | unit | 5m 12s | | tez-dag in the patch passed. |
| +1 :green_heart: | unit | 40m 53s | | tez-tests in the patch passed.
|
| +1 :green_heart: | unit | 4m 29s | | tez-ext-service-tests in the
patch passed. |
| -1 :x: | unit | 72m 54s |
[/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/18/artifact/out/patch-unit-root.txt)
| root in the patch passed. |
| +1 :green_heart: | asflicense | 2m 44s | | The patch does not
generate ASF License warnings. |
| | | 207m 7s | | |
| 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/18/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 27575df66c8f 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 / 466378a39c0fd561511f8b4953bd62ce3b3152a8 |
| Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 |
| Test Results |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/18/testReport/ |
| Max. process+thread count | 1382 (vs. ulimit of 5500) |
| modules | C: tez-api tez-runtime-library tez-dag tez-tests
tez-ext-service-tests . U: . |
| Console output |
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/18/console |
| versions | git=2.34.1 maven=3.6.3 hadolint=1.18.0-0-g76eee5c
codespell=2.0.0 shellcheck=0.7.1 yamllint=1.24.2 |
| Powered by | Apache Yetus 0.15.1 https://yetus.apache.org |
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2840973430 (!) 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/18/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2835317832
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 12m 12s | | 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 1s | | The patch appears to
include 3 new or modified test files. |
_ master Compile Tests _ |
| +0 :ok: | mvndep | 2m 32s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 28s | | master passed |
| +1 :green_heart: | compile | 3m 14s | | master passed |
| +1 :green_heart: | checkstyle | 2m 10s | | master passed |
| +1 :green_heart: | javadoc | 2m 36s | | master passed |
| -1 :x: | spotbugs | 0m 15s |
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-api.txt)
| tez-api in master failed. |
| -1 :x: | spotbugs | 0m 15s |
[/branch-spotbugs-tez-runtime-internals.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-runtime-internals.txt)
| tez-runtime-internals in master failed. |
| -1 :x: | spotbugs | 0m 15s |
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-runtime-library.txt)
| tez-runtime-library in master failed. |
| -1 :x: | spotbugs | 0m 13s |
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-dag.txt)
| tez-dag in master failed. |
| -1 :x: | spotbugs | 0m 15s |
[/branch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-tests.txt)
| tez-tests in master failed. |
| -1 :x: | spotbugs | 0m 14s |
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
| tez-ext-service-tests in master failed. |
| -1 :x: | spotbugs | 0m 15s |
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/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 48s | | the patch passed |
| +1 :green_heart: | codespell | 0m 27s | | No new issues. |
| +1 :green_heart: | compile | 3m 19s | | the patch passed |
| +1 :green_heart: | javac | 3m 19s | | the patch passed |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 14s |
[/results-checkstyle-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/results-checkstyle-tez-api.txt)
| tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was
35) |
| -0 :warning: | checkstyle | 0m 25s |
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/results-checkstyle-root.txt)
| root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was
37) |
| -1 :x: | hadolint | 0m 1s |
[/results-hadolint.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/17/artifact/out/results-hadolint.txt)
| The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) |
| +1 :green_heart: | shellcheck | 0m 2s | | No new issues. |
| +1 :green_heart: | yamllint | 0m 2s | | No new issues. |
| +1 :green_heart: | javadoc | 2m 26s | | the patch passed |
| +1 :green_heart: | spotbugs | 9m 5s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 1m 59s | | tez-api in the patch passed. |
| +1 :green_heart: | unit | 0m 43s | | tez-runtime-internals in the
patch passed. |
| +1 :green_heart: | unit | 4m 12s | | tez-runtime-library in the
patch passed. |
| +1 :green_heart: | unit | 4m 2s | | tez-dag in the patch passed. |
| +1 :green_heart: | unit | 34m 6s | | tez-tests in the patch passed.
|
| +1 :green_heart: | unit | 3m 48s | | tez-ext-service-tests in the
patch passed. |
| +1 :green_heart: | unit |
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2834887416 (!) 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/17/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407:
URL: https://github.com/apache/tez/pull/407#issuecomment-2830292452
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 41s | | 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 42s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 9m 23s | | master passed |
| +1 :green_heart: | compile | 5m 42s | | master passed |
| +1 :green_heart: | checkstyle | 3m 52s | | master passed |
| +1 :green_heart: | javadoc | 4m 7s | | master passed |
| -1 :x: | spotbugs | 0m 22s |
[/branch-spotbugs-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/branch-spotbugs-tez-api.txt)
| tez-api in master failed. |
| -1 :x: | spotbugs | 0m 21s |
[/branch-spotbugs-tez-runtime-internals.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/branch-spotbugs-tez-runtime-internals.txt)
| tez-runtime-internals in master failed. |
| -1 :x: | spotbugs | 0m 21s |
[/branch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/branch-spotbugs-tez-runtime-library.txt)
| tez-runtime-library in master failed. |
| -1 :x: | spotbugs | 0m 22s |
[/branch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/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/16/artifact/out/branch-spotbugs-tez-tests.txt)
| tez-tests in master failed. |
| -1 :x: | spotbugs | 0m 21s |
[/branch-spotbugs-tez-ext-service-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/branch-spotbugs-tez-ext-service-tests.txt)
| tez-ext-service-tests in master failed. |
| -1 :x: | spotbugs | 0m 22s |
[/branch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/branch-spotbugs-root.txt)
| root in master failed. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 17s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 6m 42s | | the patch passed |
| +1 :green_heart: | codespell | 0m 52s | | No new issues. |
| +1 :green_heart: | compile | 5m 49s | | the patch passed |
| +1 :green_heart: | javac | 5m 49s | | the patch passed |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 26s |
[/results-checkstyle-tez-api.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/results-checkstyle-tez-api.txt)
| tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was
35) |
| -0 :warning: | checkstyle | 0m 52s |
[/results-checkstyle-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/results-checkstyle-root.txt)
| root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was
37) |
| -1 :x: | hadolint | 0m 2s |
[/results-hadolint.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/results-hadolint.txt)
| The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) |
| -1 :x: | shellcheck | 0m 3s |
[/results-shellcheck.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/16/artifact/out/results-shellcheck.txt)
| The patch generated 1 new + 134 unchanged - 1 fixed = 135 total (was 135) |
| +1 :green_heart: | yamllint | 0m 0s | | No new issues. |
| +1 :green_heart: | javadoc | 3m 58s | | the patch passed |
| +1 :green_heart: | spotbugs | 18m 56s | | the patch passed |
_ Other Tests _ |
| +1 :green_heart: | unit | 2m 29s | | tez-api in the patch passed. |
| +1 :green_heart: | unit | 1m 3s | | tez-runtime-internals in the
patch passed. |
| +1 :green_heart: | unit | 6m 8s | | tez-runtime-library in the
patch passed. |
| +1 :green_heart: | unit | 5m 12s | | tez-dag in the patch passed. |
| +1 :green_heart: | unit |
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2829795425 (!) 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/16/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2829358647 (!) 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/15/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2827900771 (!) 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/14/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820861020 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 41s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 38s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 9m 22s | master passed | | +1 :green_heart: | compile | 5m 46s | master passed | | +1 :green_heart: | checkstyle | 3m 53s | master passed | | +1 :green_heart: | javadoc | 4m 7s | master passed | | +0 :ok: | spotbugs | 6m 44s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 23s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 22s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 18s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 6m 40s | the patch passed | | +1 :green_heart: | compile | 5m 48s | the patch passed | | +1 :green_heart: | javac | 5m 48s | the patch passed | | -0 :warning: | checkstyle | 0m 26s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 54s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | -1 :x: | shellcheck | 0m 3s | The patch generated 1 new + 134 unchanged - 1 fixed = 135 total (was 135) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 3m 57s | the patch passed | | +1 :green_heart: | spotbugs | 18m 45s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 29s | tez-api in the patch passed. | | +1 :green_heart: | unit | 1m 3s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 6m 7s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 5m 13s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 41m 17s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 4m 32s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 73m 30s | root in the patch passed. | | +1 :green_heart: | asflicense | 3m 8s | The patch does not generate ASF License warnings. | | | | 209m 6s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | 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/9/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 6562d3c2b642 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/9/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/9/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/9/artifact/o
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820811253 (!) 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/13/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820710255 (!) 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/12/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820704909 (!) 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/11/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-282033 (!) 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/10/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820597668 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 35s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 22s | master passed | | +1 :green_heart: | compile | 3m 15s | master passed | | +1 :green_heart: | checkstyle | 2m 7s | master passed | | +1 :green_heart: | javadoc | 2m 33s | master passed | | +0 :ok: | spotbugs | 4m 17s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 14s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 15s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 15s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 15s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 12s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 57s | the patch passed | | +1 :green_heart: | compile | 3m 11s | the patch passed | | +1 :green_heart: | javac | 3m 11s | the patch passed | | -0 :warning: | checkstyle | 0m 15s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 28s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 2m 18s | the patch passed | | +1 :green_heart: | spotbugs | 10m 11s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 58s | tez-api in the patch passed. | | +1 :green_heart: | unit | 0m 43s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 4m 21s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 3m 56s | tez-dag in the patch passed. | | -1 :x: | unit | 34m 3s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 46s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 57m 32s | root in the patch passed. | | +1 :green_heart: | asflicense | 2m 3s | The patch does not generate ASF License warnings. | | | | 152m 17s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | Failed junit tests | tez.test.TestRecovery | | | tez.test.TestRecovery | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/8/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 104a82213d0b 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/8/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/8/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/8/artifact/out/branc
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820240328 (!) 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/9/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820193059 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 42s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 1s | Shelldocs 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 :green_heart: | mvninstall | 9m 33s | master passed | | +1 :green_heart: | compile | 5m 47s | master passed | | +1 :green_heart: | checkstyle | 3m 53s | master passed | | +1 :green_heart: | javadoc | 4m 8s | master passed | | +0 :ok: | spotbugs | 6m 43s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 22s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 22s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 18s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 19s | tez-tests in the patch failed. | | +1 :green_heart: | compile | 5m 48s | the patch passed | | +1 :green_heart: | javac | 5m 48s | the patch passed | | -0 :warning: | checkstyle | 0m 27s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 52s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 3s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 0s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 3m 59s | the patch passed | | +1 :green_heart: | spotbugs | 18m 48s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 28s | tez-api in the patch passed. | | +1 :green_heart: | unit | 1m 4s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 6m 10s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 5m 12s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 42m 11s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 4m 33s | tez-ext-service-tests in the patch passed. | | +1 :green_heart: | unit | 74m 30s | root in the patch passed. | | +1 :green_heart: | asflicense | 3m 10s | The patch does not generate ASF License warnings. | | | | 211m 15s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/6/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 47f0585bbddf 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/6/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/6/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/6/artifact/out/branch-spotbugs-tez-runtime-library.txt | | spotbugs | http
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820169163 (!) 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/8/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2820161027 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 11m 3s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 33s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 30s | master passed | | +1 :green_heart: | compile | 3m 24s | master passed | | +1 :green_heart: | checkstyle | 2m 8s | master passed | | +1 :green_heart: | javadoc | 2m 29s | master passed | | +0 :ok: | spotbugs | 4m 12s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 13s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 15s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 15s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 14s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 15s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 12s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 11s | tez-tests in the patch failed. | | +1 :green_heart: | compile | 3m 16s | the patch passed | | +1 :green_heart: | javac | 3m 16s | the patch passed | | -0 :warning: | checkstyle | 0m 17s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 25s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 2m 25s | the patch passed | | +1 :green_heart: | spotbugs | 10m 13s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 54s | tez-api in the patch passed. | | +1 :green_heart: | unit | 0m 43s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 4m 16s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 4m 4s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 34m 2s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 45s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 57m 37s | root in the patch passed. | | +1 :green_heart: | asflicense | 2m 5s | The patch does not generate ASF License warnings. | | | | 163m 23s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | 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/7/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 553308603ee7 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/7/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/7/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/7/artifact/out/branch-spotbugs-tez-
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2819961437 (!) 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/7/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2819930882 (!) 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/6/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2819640815 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 44s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 40s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 9m 32s | master passed | | +1 :green_heart: | compile | 15m 28s | master passed | | +1 :green_heart: | checkstyle | 10m 32s | master passed | | +1 :green_heart: | javadoc | 11m 1s | master passed | | +0 :ok: | spotbugs | 19m 57s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 22s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-common in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-mapreduce in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-examples in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-ui in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-plugins in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-protobuf-history-plugin in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-yarn-timeline-history in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-yarn-timeline-history-with-acls in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-yarn-timeline-cache-plugin in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-yarn-timeline-history-with-fs in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-history-parser in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-aux-services in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-tools in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-javadoc-tools in master failed. | | -1 :x: | spotbugs | 0m 20s | docs in master failed. | | -1 :x: | spotbugs | 0m 21s | root in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dist in master failed. | | -1 :x: | spotbugs | 0m 22s | job-analyzer in master failed. | | -1 :x: | spotbugs | 0m 20s | tez-tfile-parser in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 19s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 19s | tez-tests in the patch failed. | | -1 :x: | mvninstall | 0m 34s | tez-tools in the patch failed. | | -1 :x: | mvninstall | 0m 24s | tez-javadoc-tools in the patch failed. | | +1 :green_heart: | compile | 14m 44s | the patch passed | | +1 :green_heart: | javac | 14m 44s | the patch passed | | -0 :warning: | checkstyle | 0m 27s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 53s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 3s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 4s | The patch has 25 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 2s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 10m 46s | the patch passed | | +0 :ok: | spotbugs | 0m 47s | tez-ui has no data from spotbugs | | +0 :ok: | spotbugs | 0m 30s | tez-dist has no data from spotbugs | | +0 :ok: | spotbugs | 0m 25s | docs has no data from spotbugs | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 29s | tez-api in the patch passed. | | +1 :green_heart: | unit | 0m 41s | tez-common in the patch passed. | | +1 :green_heart: | unit | 1m 3s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 6m 6s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 1m
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2819025150 (!) 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/5/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2817920624 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 19m 55s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 43s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 9m 40s | master passed | | +1 :green_heart: | compile | 5m 46s | master passed | | +1 :green_heart: | checkstyle | 3m 55s | master passed | | +1 :green_heart: | javadoc | 4m 7s | master passed | | +0 :ok: | spotbugs | 6m 42s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 22s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 22s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 21s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 17s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 20s | tez-tests in the patch failed. | | +1 :green_heart: | compile | 5m 46s | the patch passed | | +1 :green_heart: | javac | 5m 46s | the patch passed | | -0 :warning: | checkstyle | 0m 27s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 54s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 1s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 3m 59s | the patch passed | | +1 :green_heart: | spotbugs | 14m 43s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 28s | tez-api in the patch passed. | | +1 :green_heart: | unit | 1m 4s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 6m 11s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 5m 13s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 40m 36s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 4m 36s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 73m 27s | root in the patch passed. | | +1 :green_heart: | asflicense | 3m 8s | The patch does not generate ASF License warnings. | | | | 223m 51s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | 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/4/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 88343b143220 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/4/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/4/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/4/artifact/out/branch-spotbugs-tez-
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2817612693 (!) 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/4/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2814087089 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 42s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 37s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 9m 30s | master passed | | +1 :green_heart: | compile | 5m 48s | master passed | | +1 :green_heart: | checkstyle | 3m 53s | master passed | | +1 :green_heart: | javadoc | 4m 6s | master passed | | +0 :ok: | spotbugs | 6m 40s | Both FindBugs and SpotBugs are enabled, using SpotBugs. | | -1 :x: | spotbugs | 0m 22s | tez-api in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-internals in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-runtime-library in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-dag in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-tests in master failed. | | -1 :x: | spotbugs | 0m 21s | tez-ext-service-tests in master failed. | | -1 :x: | spotbugs | 0m 22s | root in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 15s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 6m 35s | the patch passed | | +1 :green_heart: | compile | 5m 45s | the patch passed | | +1 :green_heart: | javac | 5m 45s | the patch passed | | -0 :warning: | checkstyle | 0m 27s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 53s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 3s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 2s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 3m 57s | the patch passed | | +1 :green_heart: | spotbugs | 15m 59s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 26s | tez-api in the patch passed. | | +1 :green_heart: | unit | 1m 2s | tez-runtime-internals in the patch passed. | | +1 :green_heart: | unit | 6m 10s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 5m 11s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 40m 36s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 4m 29s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 72m 41s | root in the patch passed. | | +1 :green_heart: | asflicense | 3m 8s | The patch does not generate ASF License warnings. | | | | 204m 36s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-api/pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-runtime-internals/pom.xml | | | tez-tests/pom.xml | | 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/3/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux b3951a2a623d 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/3/artifact/out/branch-spotbugs-tez-api.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/3/artifact/out/branch-spotbugs-tez-runtime-internals.txt | | spotbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/3/artifact/out/branch-spotbugs-tez-runt
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2813725722 (!) 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/3/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2812458594 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 14m 55s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 1s | Shelldocs 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 24s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 50s | master passed | | +1 :green_heart: | compile | 3m 12s | master passed | | +1 :green_heart: | checkstyle | 1m 56s | master passed | | +1 :green_heart: | javadoc | 2m 0s | master passed | | +0 :ok: | spotbugs | 3m 49s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 45s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 8s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 4m 44s | the patch passed | | +1 :green_heart: | compile | 3m 20s | the patch passed | | +1 :green_heart: | javac | 3m 20s | the patch passed | | -0 :warning: | checkstyle | 0m 17s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 32s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 1s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 0s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 2m 26s | the patch passed | | -1 :x: | findbugs | 0m 29s | tez-api in the patch failed. | | -1 :x: | findbugs | 0m 27s | tez-runtime-library in the patch failed. | | -1 :x: | findbugs | 0m 30s | tez-dag in the patch failed. | | -1 :x: | findbugs | 0m 19s | tez-tests in the patch failed. | | -1 :x: | findbugs | 0m 22s | tez-ext-service-tests in the patch failed. | | -1 :x: | findbugs | 0m 18s | root in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 58s | tez-api in the patch passed. | | +1 :green_heart: | unit | 4m 2s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 3m 51s | tez-dag in the patch passed. | | -1 :x: | unit | 33m 56s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 42s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 58m 43s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 41s | The patch does not generate ASF License warnings. | | | | 164m 22s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Failed junit tests | tez.test.TestRecovery | | | tez.test.TestRecovery | | | tez.analyzer.TestAnalyzer | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/407 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 1d98f5b08d54 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/artifact/out/diff-patch-hadolint.txt | | xml | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/artifact/out/xml.txt | | findbugs | https://ci-hadoop.apache.org/
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2812542079 (!) 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/2/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #407: URL: https://github.com/apache/tez/pull/407#issuecomment-2812088386 (!) 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/1/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: Migrating to JDK17 [tez]
maheshrajus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2812088638 Raised new PR: 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: Migrating to JDK17 [tez]
maheshrajus closed pull request #405: Tez: Migrating to JDK17 URL: https://github.com/apache/tez/pull/405 -- 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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2809029443 (!) 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-405/21/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2808924400 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 32s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 43s | master passed | | +1 :green_heart: | compile | 2m 58s | master passed | | +1 :green_heart: | checkstyle | 2m 3s | master passed | | +1 :green_heart: | javadoc | 2m 26s | master passed | | +0 :ok: | spotbugs | 3m 23s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 19s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | -1 :x: | mvninstall | 0m 5s | root in master failed. | | +0 :ok: | mvndep | 0m 9s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 5s | root in the patch failed. | | -1 :x: | mvninstall | 0m 5s | tez-api in the patch failed. | | -1 :x: | mvninstall | 0m 4s | tez-dag in the patch failed. | | -1 :x: | mvninstall | 0m 5s | tez-ext-service-tests in the patch failed. | | -1 :x: | mvninstall | 0m 4s | tez-runtime-library in the patch failed. | | -1 :x: | mvninstall | 0m 4s | tez-tests in the patch failed. | | -1 :x: | compile | 0m 4s | root in the patch failed. | | -1 :x: | compile | 0m 4s | tez-api in the patch failed. | | -1 :x: | compile | 0m 5s | tez-dag in the patch failed. | | -1 :x: | compile | 0m 4s | tez-ext-service-tests in the patch failed. | | -1 :x: | compile | 0m 4s | tez-runtime-library in the patch failed. | | -1 :x: | compile | 0m 4s | tez-tests in the patch failed. | | -1 :x: | javac | 0m 4s | root in the patch failed. | | -1 :x: | javac | 0m 4s | tez-api in the patch failed. | | -1 :x: | javac | 0m 5s | tez-dag in the patch failed. | | -1 :x: | javac | 0m 4s | tez-ext-service-tests in the patch failed. | | -1 :x: | javac | 0m 4s | tez-runtime-library in the patch failed. | | -1 :x: | javac | 0m 4s | tez-tests in the patch failed. | | -0 :warning: | checkstyle | 0m 3s | The patch fails to run checkstyle in root | | -0 :warning: | checkstyle | 0m 1s | The patch fails to run checkstyle in tez-api | | -0 :warning: | checkstyle | 0m 2s | The patch fails to run checkstyle in tez-dag | | -0 :warning: | checkstyle | 0m 2s | The patch fails to run checkstyle in tez-ext-service-tests | | -0 :warning: | checkstyle | 0m 2s | The patch fails to run checkstyle in tez-runtime-library | | -0 :warning: | checkstyle | 0m 2s | The patch fails to run checkstyle in tez-tests | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | -1 :x: | javadoc | 0m 4s | root in the patch failed. | | -1 :x: | javadoc | 0m 4s | tez-api in the patch failed. | | -1 :x: | javadoc | 0m 5s | tez-dag in the patch failed. | | -1 :x: | javadoc | 0m 4s | tez-ext-service-tests in the patch failed. | | -1 :x: | javadoc | 0m 5s | tez-runtime-library in the patch failed. | | -1 :x: | javadoc | 0m 4s | tez-tests in the patch failed. | | -1 :x: | findbugs | 0m 5s | root in the patch failed. | | -1 :x: | findbugs | 0m 5s | tez-api in the patch failed. | | -1 :x: | findbugs | 0m 4s | tez-dag in the patch failed. | | -1 :x: | findbugs | 0m 4s | tez-ext-service-tests in the patch failed. | | -1 :x: | findbugs | 0m 4s | tez-runtime-library in the patch failed. | | -1 :x: | findbugs | 0m 4s | tez-tests in the patch failed. | ||| _ Other Tests _ | | -1 :x: | unit | 0m 5s | root in the patch failed. | | -1 :x: | unit | 0m 5s | tez-api in the patch failed. | | -1 :x: | unit | 0m 4s | tez-dag in the patch failed. | | -1
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2808846427 (!) 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-405/20/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2804178250 (!) 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-405/19/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2801740686 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 34s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 51s | master passed | | +1 :green_heart: | compile | 3m 21s | master passed | | +1 :green_heart: | checkstyle | 2m 9s | master passed | | +1 :green_heart: | javadoc | 2m 22s | master passed | | +0 :ok: | spotbugs | 3m 6s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 1s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 10s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 13s | tez-tests in the patch failed. | | +1 :green_heart: | compile | 3m 16s | the patch passed | | +1 :green_heart: | javac | 3m 16s | the patch passed | | -0 :warning: | checkstyle | 0m 13s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 31s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 2m 9s | the patch passed | | -1 :x: | findbugs | 0m 32s | tez-api cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 28s | tez-runtime-library cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 33s | tez-dag cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 27s | tez-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 28s | tez-ext-service-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 1m 30s | root cannot run convertXmlToText from findbugs | ||| _ Other Tests _ | | +1 :green_heart: | unit | 2m 1s | tez-api in the patch passed. | | +1 :green_heart: | unit | 4m 33s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 4m 12s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 32m 5s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 52s | tez-ext-service-tests in the patch passed. | | +1 :green_heart: | unit | 59m 15s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 49s | The patch does not generate ASF License warnings. | | | | 149m 5s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/18/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/405 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 8e5d288cd79e 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 | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | mvninstall | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/18/artifact/out/patch-mvninstall-tez-tests.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/18/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/18/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/18/artifact/out/diff
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2801346751 (!) 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-405/18/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2796407664 (!) 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-405/15/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2796170812 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 9s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 25s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 25s | master passed | | +1 :green_heart: | compile | 2m 51s | master passed | | +1 :green_heart: | checkstyle | 1m 40s | master passed | | +1 :green_heart: | javadoc | 1m 49s | master passed | | +0 :ok: | spotbugs | 3m 8s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 4s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 9s | Maven dependency ordering for patch | | -1 :x: | mvninstall | 0m 11s | tez-tests in the patch failed. | | +1 :green_heart: | compile | 3m 3s | the patch passed | | +1 :green_heart: | javac | 3m 3s | the patch passed | | -0 :warning: | checkstyle | 0m 15s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 26s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 1s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 0s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 1m 56s | the patch passed | | -1 :x: | findbugs | 0m 27s | tez-api cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 25s | tez-runtime-library cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 29s | tez-dag cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 25s | tez-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 24s | tez-ext-service-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 1m 27s | root cannot run convertXmlToText from findbugs | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 59s | tez-api in the patch passed. | | +1 :green_heart: | unit | 4m 26s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 4m 9s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 34m 35s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 43s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 58m 44s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 48s | The patch does not generate ASF License warnings. | | | | 146m 41s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Failed junit tests | tez.analyzer.TestAnalyzer | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/14/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/405 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux aed375d6208d 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | mvninstall | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/14/artifact/out/patch-mvninstall-tez-tests.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/14/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/14/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2796128037 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 35s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 32s | master passed | | +1 :green_heart: | compile | 2m 55s | master passed | | +1 :green_heart: | checkstyle | 1m 52s | master passed | | +1 :green_heart: | javadoc | 2m 17s | master passed | | +0 :ok: | spotbugs | 3m 16s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 12s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 8s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 38s | the patch passed | | +1 :green_heart: | compile | 2m 52s | the patch passed | | +1 :green_heart: | javac | 2m 52s | the patch passed | | -0 :warning: | checkstyle | 0m 12s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 23s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 0s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 1m 42s | the patch passed | | -1 :x: | findbugs | 0m 25s | tez-api cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 23s | tez-runtime-library cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 29s | tez-dag cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 25s | tez-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 22s | tez-ext-service-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 1m 22s | root cannot run convertXmlToText from findbugs | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 47s | tez-api in the patch passed. | | +1 :green_heart: | unit | 3m 59s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 3m 44s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 33m 52s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 40s | tez-ext-service-tests in the patch passed. | | +1 :green_heart: | unit | 58m 41s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 37s | The patch does not generate ASF License warnings. | | | | 145m 7s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/13/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/405 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 1555e59f9681 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/13/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/13/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/13/artifact/out/diff-patch-hadolint.txt | | xml | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/13/artifact/out/xml.txt | | findbu
Re: [PR] Tez: Migrating to JDK17 [tez]
Aggarwal-Raghav commented on code in PR #405: URL: https://github.com/apache/tez/pull/405#discussion_r2038872092 ## pom.xml: ## @@ -926,7 +932,13 @@ 1 false 900 --Xmx1024m -XX:+HeapDumpOnOutOfMemoryError +-Xmx4096m -XX:+HeapDumpOnOutOfMemoryError + + --add-opens=java.base/java.lang=ALL-UNNAMED + --add-opens=java.base/java.util=ALL-UNNAMED Review Comment: Consolidation of all "--add-opens" in a single variable and refer to it in other module pom would be better. -- 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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2795890076 (!) 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-405/14/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2795867916 (!) 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-405/13/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: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2792741727 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 26s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 25s | master passed | | +1 :green_heart: | compile | 2m 50s | master passed | | +1 :green_heart: | checkstyle | 1m 42s | master passed | | +1 :green_heart: | javadoc | 1m 55s | master passed | | +0 :ok: | spotbugs | 3m 18s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 15s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 8s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 50s | the patch passed | | +1 :green_heart: | compile | 2m 48s | the patch passed | | +1 :green_heart: | javac | 2m 48s | the patch passed | | -0 :warning: | checkstyle | 0m 12s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 25s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 1s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 1s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 1m 43s | the patch passed | | -1 :x: | findbugs | 0m 25s | tez-api cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 25s | tez-runtime-library cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 28s | tez-dag cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 25s | tez-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 0m 26s | tez-ext-service-tests cannot run convertXmlToText from findbugs | | -1 :x: | findbugs | 1m 23s | root cannot run convertXmlToText from findbugs | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 50s | tez-api in the patch passed. | | +1 :green_heart: | unit | 4m 17s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 4m 6s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 34m 26s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 35s | tez-ext-service-tests in the patch passed. | | -1 :x: | unit | 61m 29s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 48s | The patch does not generate ASF License warnings. | | | | 149m 44s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Failed junit tests | tez.analyzer.TestAnalyzer | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/12/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/405 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux 3988e4089be8 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/12/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/12/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/12/artifact/out/diff-patch-hadolint.txt | | xml | https://ci-hadoop.apache.org/job/tez-multibranch/job/P
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2792689348 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 3m 27s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs 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 33s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 34s | master passed | | +1 :green_heart: | compile | 3m 3s | master passed | | +1 :green_heart: | checkstyle | 1m 43s | master passed | | +1 :green_heart: | javadoc | 1m 52s | master passed | | +0 :ok: | spotbugs | 3m 21s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +0 :ok: | findbugs | 3m 18s | root in master has 4 extant findbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 8s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 53s | the patch passed | | +1 :green_heart: | compile | 2m 56s | the patch passed | | +1 :green_heart: | javac | 2m 56s | the patch passed | | -0 :warning: | checkstyle | 0m 12s | tez-api: The patch generated 3 new + 35 unchanged - 0 fixed = 38 total (was 35) | | -0 :warning: | checkstyle | 0m 23s | root: The patch generated 3 new + 37 unchanged - 0 fixed = 40 total (was 37) | | -1 :x: | hadolint | 0m 2s | The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | -1 :x: | xml | 0m 0s | The patch has 4 ill-formed XML file(s). | | +1 :green_heart: | yamllint | 0m 0s | There were no new yamllint issues. | | +1 :green_heart: | javadoc | 1m 43s | the patch passed | | -1 :x: | findbugs | 0m 20s | tez-api in the patch failed. | | -1 :x: | findbugs | 0m 19s | tez-runtime-library in the patch failed. | | -1 :x: | findbugs | 0m 23s | tez-dag in the patch failed. | | -1 :x: | findbugs | 0m 18s | tez-tests in the patch failed. | | -1 :x: | findbugs | 0m 18s | tez-ext-service-tests in the patch failed. | | -1 :x: | findbugs | 0m 15s | root in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 46s | tez-api in the patch passed. | | +1 :green_heart: | unit | 4m 14s | tez-runtime-library in the patch passed. | | +1 :green_heart: | unit | 3m 54s | tez-dag in the patch passed. | | +1 :green_heart: | unit | 32m 31s | tez-tests in the patch passed. | | +1 :green_heart: | unit | 3m 47s | tez-ext-service-tests in the patch passed. | | +1 :green_heart: | unit | 60m 49s | root in the patch passed. | | +1 :green_heart: | asflicense | 1m 37s | The patch does not generate ASF License warnings. | | | | 149m 9s | | | Reason | Tests | |---:|:--| | XML | Parsing Error(s): | | | pom.xml | | | tez-dag/pom.xml | | | tez-ext-service-tests/pom.xml | | | tez-tests/pom.xml | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/tez/pull/405 | | Optional Tests | dupname asflicense hadolint shellcheck shelldocs yamllint javac javadoc unit xml compile spotbugs findbugs checkstyle | | uname | Linux cc786465fe73 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | personality/tez.sh | | git revision | master / 6d25c3ce0 | | Default Java | Ubuntu-17.0.14+7-Ubuntu-122.04.1 | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/diff-checkstyle-tez-api.txt | | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/diff-checkstyle-root.txt | | hadolint | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/diff-patch-hadolint.txt | | xml | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/xml.txt | | findbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-405/11/artifact/out/patch-findbugs-tez-api.txt | | findbugs | h
Re: [PR] Tez: Migrating to JDK17 [tez]
tez-yetus commented on PR #405: URL: https://github.com/apache/tez/pull/405#issuecomment-2792307304 (!) 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-405/12/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
