Skip to content

TEZ-4310: Avoid duplicated precommit comments - #176

Merged
jteagles merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4310
Jan 21, 2022
Merged

TEZ-4310: Avoid duplicated precommit comments#176
jteagles merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4310

Conversation

@abstractdog

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec12m 53sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚shellcheck0m 0sThere were no new shellcheck issues.
+1 💚whitespace0m 1sThe patch has no whitespace issues.
_ Other Tests _
+1 💚asflicense1m 2sThe patch does not generate ASF License warnings.
14m 38s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-176/1/artifact/out/Dockerfile
GITHUB PR#176
JIRA IssueTEZ-4310
Optional Testsdupname asflicense shellcheck shelldocs
unameLinux be2f097da1eb 4.15.0-161-generic #169-Ubuntu SMP Fri Oct 15 13:41:54 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 41cbc17
Max. process+thread count51 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-176/1/console
versionsgit=2.25.1 maven=3.6.3 shellcheck=0.7.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
ContributorAuthor

can you approve this change @jteagles? looks like it's working:

  1. the comment made by tez-yetus hasn't gone back to jira (no duplicates)
  2. the jira has kept successfully linked to this PR

@jteaglesjteagles left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@jteagles
jteagles merged commit 0210dab into apache:masterJan 21, 2022
@jteagles

Copy link
Copy Markdown
Contributor

Does this need to go to branch-0.9?

@abstractdog

abstractdog commented Jan 23, 2022

Copy link
Copy Markdown
ContributorAuthor

let me check, I'm opening a PR against branch-0.9
#179

abstractdog added a commit to abstractdog/tez that referenced this pull request Jan 23, 2022
Sign up for freeto 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

@abstractdog@tez-yetus@jteagles