Skip to content

Conversation

@abstractdog
Copy link
Contributor

No description provided.

@abstractdog abstractdog requested a review from ayushtkn May 7, 2025 09:39
@tez-yetus

This comment was marked as outdated.

@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 3m 24s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗 mvndep 2m 50s Maven dependency ordering for branch
+1 💚 mvninstall 9m 29s master passed
+1 💚 compile 18m 18s master passed with JDK Ubuntu-11.0.27+6-post-Ubuntu-0ubuntu122.04
+1 💚 compile 17m 39s master passed with JDK Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
+1 💚 javadoc 13m 11s master passed with JDK Ubuntu-11.0.27+6-post-Ubuntu-0ubuntu122.04
+1 💚 javadoc 12m 40s master passed with JDK Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
_ Patch Compile Tests _
+1 💚 mvninstall 4m 11s master passed
+0 🆗 mvndep 4m 39s Maven dependency ordering for patch
+1 💚 mvninstall 21m 50s the patch passed
+1 💚 codespell 0m 53s No new issues.
+1 💚 compile 18m 22s the patch passed with JDK Ubuntu-11.0.27+6-post-Ubuntu-0ubuntu122.04
+1 💚 javac 18m 22s the patch passed
+1 💚 compile 17m 37s the patch passed with JDK Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
+1 💚 javac 17m 37s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 javadoc 13m 4s the patch passed with JDK Ubuntu-11.0.27+6-post-Ubuntu-0ubuntu122.04
+1 💚 javadoc 12m 41s the patch passed with JDK Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
_ Other Tests _
+1 💚 unit 0m 31s hadoop-shim in the patch passed.
+1 💚 unit 2m 20s tez-api in the patch passed.
+1 💚 unit 0m 21s tez-build-tools in the patch passed.
+1 💚 unit 0m 39s tez-common in the patch passed.
+1 💚 unit 1m 2s tez-runtime-internals in the patch passed.
+1 💚 unit 5m 58s tez-runtime-library in the patch passed.
+1 💚 unit 1m 35s tez-mapreduce in the patch passed.
+1 💚 unit 0m 28s tez-examples in the patch passed.
+1 💚 unit 5m 19s tez-dag in the patch passed.
+1 💚 unit 44m 50s tez-tests in the patch passed.
+1 💚 unit 4m 51s tez-ext-service-tests in the patch passed.
+1 💚 unit 1m 38s tez-ui in the patch passed.
+1 💚 unit 10m 0s tez-plugins in the patch passed.
+1 💚 unit 0m 43s tez-protobuf-history-plugin in the patch passed.
+1 💚 unit 1m 52s tez-yarn-timeline-history in the patch passed.
+1 💚 unit 2m 6s tez-yarn-timeline-history-with-acls in the patch passed.
+1 💚 unit 0m 30s tez-yarn-timeline-cache-plugin in the patch passed.
+1 💚 unit 1m 50s tez-yarn-timeline-history-with-fs in the patch passed.
+1 💚 unit 2m 46s tez-history-parser in the patch passed.
+1 💚 unit 3m 10s tez-aux-services in the patch passed.
-1 ❌ unit 3m 4s /patch-unit-tez-tools.txt tez-tools in the patch passed.
-1 ❌ unit 2m 59s /patch-unit-tez-tools_analyzers.txt analyzers in the patch failed.
-1 ❌ unit 3m 6s /patch-unit-tez-tools_analyzers_job-analyzer.txt job-analyzer in the patch failed.
+1 💚 unit 0m 31s tez-javadoc-tools in the patch passed.
+1 💚 unit 0m 33s hadoop-shim-impls in the patch passed.
+1 💚 unit 0m 32s hadoop-shim-2.8 in the patch passed.
+1 💚 unit 0m 32s tez-dist in the patch passed.
+1 💚 unit 0m 26s docs in the patch passed.
-1 ❌ unit 80m 38s /patch-unit-root.txt root in the patch failed.
+1 💚 unit 0m 31s hadoop-shim-2.7 in the patch passed.
+1 💚 unit 0m 32s tez-tfile-parser in the patch passed.
+1 💚 asflicense 13m 31s The patch does not generate ASF License warnings.
370m 55s
Reason Tests
Failed junit tests tez.analyzer.TestAnalyzer
tez.analyzer.TestAnalyzer
tez.analyzer.TestAnalyzer
tez.analyzer.TestAnalyzer
Subsystem Report/Notes
Docker ClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-412/2/artifact/out/Dockerfile
GITHUB PR #412
Optional Tests dupname asflicense javac javadoc unit codespell detsecrets xmllint compile
uname Linux e2e06d403fb6 5.15.0-134-generic #145-Ubuntu SMP Wed Feb 12 20:08:39 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-412/src/.yetus/personality.sh
git revision master / 1494cb1
Default Java Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
Multi-JDK versions /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.27+6-post-Ubuntu-0ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_452-8u452-gaus1-0ubuntu122.04-b09
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-412/2/testReport/
Max. process+thread count 2094 (vs. ulimit of 5500)
modules C: hadoop-shim tez-api tez-build-tools tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-examples tez-dag tez-tests tez-ext-service-tests tez-ui tez-plugins tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history tez-plugins/tez-yarn-timeline-history-with-acls tez-plugins/tez-yarn-timeline-cache-plugin tez-plugins/tez-yarn-timeline-history-with-fs tez-plugins/tez-history-parser tez-plugins/tez-aux-services tez-tools tez-tools/analyzers tez-tools/analyzers/job-analyzer tez-tools/tez-javadoc-tools hadoop-shim-impls hadoop-shim-impls/hadoop-shim-2.8 tez-dist docs . hadoop-shim-impls/hadoop-shim-2.7 tez-tools/tez-tfile-parser U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-412/2/console
versions git=2.34.1 maven=3.6.3 codespell=2.0.0
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abstractdog
Copy link
Contributor Author

TestAnalyzer is a known flakiness, merging this

@abstractdog abstractdog merged commit 24dba8d into apache:master May 8, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants