Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete intermediate attempt data for failed attempts for Shuffle Handler #184

Merged
merged 1 commit into from
Feb 7, 2022

Conversation

shameersss1
Copy link
Contributor

No description provided.

…uffle Handler (apache#72) (Syed Shameerur Rahman reviewed by Laszlo Bodor)
@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 16m 47s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 2 new or modified test files.
_ branch-0.9 Compile Tests _
+0 🆗 mvndep 2m 51s Maven dependency ordering for branch
+1 💚 mvninstall 9m 15s branch-0.9 passed
+1 💚 compile 2m 42s branch-0.9 passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 compile 2m 31s branch-0.9 passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 checkstyle 2m 36s branch-0.9 passed
+1 💚 javadoc 2m 45s branch-0.9 passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javadoc 2m 29s branch-0.9 passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+0 🆗 spotbugs 0m 57s Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚 findbugs 4m 44s branch-0.9 passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 9s Maven dependency ordering for patch
+1 💚 mvninstall 1m 35s the patch passed
+1 💚 compile 1m 34s the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javac 1m 34s the patch passed
+1 💚 compile 1m 21s the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 javac 1m 21s the patch passed
-0 ⚠️ checkstyle 0m 11s tez-plugins/tez-aux-services: The patch generated 4 new + 69 unchanged - 0 fixed = 73 total (was 69)
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 javadoc 1m 28s the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04
+1 💚 javadoc 1m 21s the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
+1 💚 findbugs 3m 53s the patch passed
_ Other Tests _
+1 💚 unit 1m 55s tez-api in the patch passed.
+1 💚 unit 0m 26s tez-common in the patch passed.
+1 💚 unit 4m 44s tez-runtime-library in the patch passed.
+1 💚 unit 4m 11s tez-dag in the patch passed.
+1 💚 unit 2m 21s tez-aux-services in the patch passed.
+1 💚 asflicense 0m 50s The patch does not generate ASF License warnings.
75m 38s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-184/1/artifact/out/Dockerfile
GITHUB PR #184
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
uname Linux 51ca474930f1 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision branch-0.9 / d8585bb
Default Java Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
Multi-JDK versions /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07
checkstyle https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-184/1/artifact/out/diff-checkstyle-tez-plugins_tez-aux-services.txt
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-184/1/testReport/
Max. process+thread count 1311 (vs. ulimit of 5500)
modules C: tez-api tez-common tez-runtime-library tez-dag tez-plugins/tez-aux-services U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-184/1/console
versions git=2.25.1 maven=3.6.3 findbugs=3.0.1
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@abstractdog abstractdog left a comment

Choose a reason for hiding this comment

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

+1

@abstractdog abstractdog merged commit 5f4c3c8 into apache:branch-0.9 Feb 7, 2022
@abstractdog
Copy link
Contributor

merged to branch-0.9, thanks @shameersss1 for the patch!

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