tez-yetus commented on PR #493: URL: https://github.com/apache/tez/pull/493#issuecomment-5286477391
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |:----:|----------:|--------:|:--------:|:-------:| | +0 :ok: | reexec | 0m 31s | | Docker mode activated. | |||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 1s | | No case conflicting files found. | | +0 :ok: | detsecrets | 0m 1s | | detect-secrets was not available. | | +0 :ok: | 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 34 new or modified test files. | |||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 43s | | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 4m 48s | | master passed | | +1 :green_heart: | compile | 4m 1s | | master passed | | +1 :green_heart: | checkstyle | 6m 12s | | master passed | | +1 :green_heart: | javadoc | 6m 31s | | master passed | | +0 :ok: | spotbugs | 1m 21s | | tez-api in master has 92 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 47s | | tez-common in master has 13 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 53s | | tez-runtime-internals in master has 92 extant spotbugs warnings. | | +0 :ok: | spotbugs | 1m 13s | | tez-runtime-library in master has 143 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 55s | | tez-mapreduce in master has 97 extant spotbugs warnings. | | +0 :ok: | spotbugs | 1m 41s | | tez-dag in master has 537 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 50s | | tez-tests in master has 6 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 49s | | tez-plugins/tez-protobuf-history-plugin in master has 7 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 43s | | tez-plugins/tez-yarn-timeline-history-with-fs in master has 5 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 44s | | tez-plugins/tez-aux-services in master has 5 extant spotbugs warnings. | | +0 :ok: | spotbugs | 6m 51s | | root in master has 1054 extant spotbugs warnings. | |||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 15s | | Maven dependency ordering for patch | | -1 :x: | mvninstall | 2m 9s | [/patch-mvninstall-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-mvninstall-root.txt) | root in the patch failed. | | +1 :green_heart: | codespell | 1m 46s | | No new issues. | | -1 :x: | compile | 1m 24s | [/patch-compile-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-compile-root.txt) | root in the patch failed. | | -1 :x: | javac | 1m 24s | [/patch-compile-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-compile-root.txt) | root in the patch failed. | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | checkstyle | 5m 58s | | the patch passed | | +1 :green_heart: | javadoc | 6m 23s | | the patch passed | | -1 :x: | spotbugs | 0m 34s | [/patch-spotbugs-tez-runtime-library.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-spotbugs-tez-runtime-library.txt) | tez-runtime-library in the patch failed. | | -1 :x: | spotbugs | 0m 41s | [/patch-spotbugs-tez-dag.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-spotbugs-tez-dag.txt) | tez-dag in the patch failed. | | -1 :x: | spotbugs | 0m 31s | [/patch-spotbugs-tez-tests.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-spotbugs-tez-tests.txt) | tez-tests in the patch failed. | | -1 :x: | spotbugs | 1m 2s | [/new-spotbugs-tez-plugins_tez-protobuf-history-plugin.html](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/new-spotbugs-tez-plugins_tez-protobuf-history-plugin.html) | tez-plugins/tez-protobuf-history-plugin generated 4 new + 3 unchanged - 4 fixed = 7 total (was 7) | | -1 :x: | spotbugs | 0m 27s | [/patch-spotbugs-tez-plugins_tez-yarn-timeline-history-with-fs.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-spotbugs-tez-plugins_tez-yarn-timeline-history-with-fs.txt) | tez-yarn-timeline-history-with-fs in the patch failed. | | -1 :x: | spotbugs | 2m 8s | [/patch-spotbugs-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-spotbugs-root.txt) | root in the patch failed. | |||| _ Other Tests _ | | -1 :x: | unit | 5m 22s | [/patch-unit-root.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/patch-unit-root.txt) | root in the patch failed. | | +1 :green_heart: | asflicense | 4m 48s | | The patch does not generate ASF License warnings. | | | | 85m 56s | | | | Reason | Tests | |-------:|:------| | SpotBugs | module:tez-plugins/tez-protobuf-history-plugin | | | Exception thrown in class org.apache.tez.dag.history.logging.proto.DatePartitionedLogger at new org.apache.tez.dag.history.logging.proto.DatePartitionedLogger(Parser, Path, Configuration, Clock) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At DatePartitionedLogger.java:at new org.apache.tez.dag.history.logging.proto.DatePartitionedLogger(Parser, Path, Configuration, Clock) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At DatePartitionedLogger.java:[line 68] | | | org.apache.tez.dag.history.logging.proto.ProtoMessageWritable.getMessage() may expose internal representation by returning ProtoMessageWritable.message At ProtoMessageWritable.java:by returning ProtoMessageWritable.message At ProtoMessageWritable.java:[line 48] | | | org.apache.tez.dag.history.logging.proto.ProtoMessageWritable.setMessage(MessageLite) may expose internal representation by storing an externally mutable object into ProtoMessageWritable.message At ProtoMessageWritable.java:by storing an externally mutable object into ProtoMessageWritable.message At ProtoMessageWritable.java:[line 52] | | | Exception thrown in class org.apache.tez.dag.history.logging.proto.ProtoMessageWriter at new org.apache.tez.dag.history.logging.proto.ProtoMessageWriter(Configuration, Path, Parser) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At ProtoMessageWriter.java:at new org.apache.tez.dag.history.logging.proto.ProtoMessageWriter(Configuration, Path, Parser) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At ProtoMessageWriter.java:[line 41] | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/artifact/out/Dockerfile | | Optional Tests | dupname compile unit asflicense codespell detsecrets javac javadoc xmllint spotbugs checkstyle | | uname | Linux a13b9698642c 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | tez-personality.sh | | git revision | master / d348bf1f18a4fac1f3fc38710131b8df7a18be1a | | Default Java | Eclipse Adoptium-21.0.11+10-LTS | | Test Results | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/testReport/ | | Max. process+thread count | 264 (vs. ulimit of 5500) | | modules | C: tez-api tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-dag tez-tests tez-ext-service-tests tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history-with-fs tez-plugins/tez-aux-services . U: . | | Console output | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-493/12/console | | versions | git=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1 | | 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
