8000 TEZ-4631: Include an official script that installs hadoop and tez andruns a simple example DAG - addendum ASF license + shellcheck fixes by abstractdog · Pull Request #417 · apache/tez · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

TEZ-4631: Include an official script that installs hadoop and tez andruns a simple example DAG - addendum ASF license + shellcheck fixes #417

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

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

abstractdog
Copy link
Contributor

No description provided.

@abstractdog abstractdog requested a review from ayushtkn June 3, 2025 08:35
@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 26m 33s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 2m 4s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 9s Maven dependency ordering for patch
+1 💚 codespell 0m 4s No new issues.
+1 💚 blanks 0m 0s The patch has no blanks issues.
-1 ❌ shellcheck 0m 0s /results-shellcheck.txt The patch generated 24 new + 0 unchanged - 0 fixed = 24 total (was 0)
_ Other Tests _
+0 🆗 asflicense 0m 0s ASF License check generated no output?
29m 9s
Subsystem Report/Notes
Docker ClientAPI=1.50 ServerAPI=1.50 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/1/artifact/out/Dockerfile
GITHUB PR #417
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs
uname Linux 2105b864b3ee 5.15.0-139-generic #149-Ubuntu SMP Fri Apr 11 22:06:13 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-417/src/.yetus/personality.sh
git revision master / e0656fe
Max. process+thread count 58 (vs. ulimit of 5500)
modules C: U:
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/1/console
versions git=2.34.1 maven=3.6.3 codespell=2.0.0 shellcheck=0.7.1
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog
Copy link
Contributor Author

LOL, shellcheck passed in the original PR #414
anyway, I'm trying to fix these if possible

@abstractdog abstractdog changed the title TEZ-4631: Include an official script that installs hadoop and tez andruns a simple example DAG - addendum ASF license TEZ-4631: Include an official script that installs hadoop and tez andruns a simple example DAG - addendum ASF license + shellcheck fixes Jun 3, 2025
@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 49s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 2m 7s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 9s Maven dependency ordering for patch
+1 💚 codespell 0m 5s No new issues.
+1 💚 blanks 0m 0s The patch has no blanks issues.
-1 ❌ shellcheck 0m 1s /results-shellcheck.txt The patch generated 23 new + 0 unchanged - 0 fixed = 23 total (was 0)
_ Other Tests _
+0 🆗 asflicense 0m 0s ASF License check generated no output?
4m 0s
Subsystem Report/Notes
Docker ClientAPI=1.50 ServerAPI=1.50 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/2/artifact/out/Dockerfile
GITHUB PR #417
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs
uname Linux 0ea9bc6c6f37 5.15.0-139-generic #149-Ubuntu SMP Fri Apr 11 22:06:13 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-417/src/.yetus/personality.sh
git revision master / 5be0073
Max. process+thread count 56 (vs. ulimit of 5500)
modules C: U:
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/2/console
versions git=2.34.1 maven=3.6.3 codespell=2.0.0 shellcheck=0.7.1
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

This comment was marked as outdated.

… runs a simple example DAG (apache#414) - addendum ASF license
@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 32s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 1m 58s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 9s Maven dependency ordering for patch
+1 💚 codespell 0m 6s No new issues.
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 shellcheck 0m 0s No new issues.
_ Other Tests _
+0 🆗 asflicense 0m 0s ASF License check generated no output?
3m 4s
Subsystem Report/Notes
Docker ClientAPI=1.50 ServerAPI=1.50 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/4/artifact/out/Dockerfile
GITHUB PR #417
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs
uname Linux 5ffa699bcfa8 5.15.0-139-generic #149-Ubuntu SMP Fri Apr 11 22:06:13 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-417/src/.yetus/personality.sh
git revision master / 6ea0d62
Max. process+thread count 55 (vs. ulimit of 5500)
modules C: U:
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-417/4/console
versions git=2.34.1 maven=3.6.3 codespell=2.0.0 shellcheck=0.7.1
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog
Copy link
Contributor Author

@ayushtkn : can you pls check? this is green now

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 abstractdog merged commit b0a65ec into apache:master Jun 4, 2025
2 checks passed
@abstractdog
Copy link
Contributor Author

thanks a lot @ayushtkn for reviewing this one!

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
0