Skip to content

Conversation

@abstractdog
Copy link
Contributor

No description provided.

@jteagles
Copy link
Contributor

+1. Seems good to me.

@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 4s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+1 💚 mvninstall 13m 10s master passed
+1 💚 mvnsite 0m 18s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 0m 8s the patch passed
+1 💚 mvnsite 0m 7s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 xml 0m 1s The patch has no ill-formed XML file.
_ Other Tests _
+1 💚 asflicense 0m 19s The patch does not generate ASF License warnings.
15m 41s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-148/1/artifact/out/Dockerfile
GITHUB PR #148
JIRA Issue TEZ-4332
Optional Tests dupname asflicense mvnsite xml
uname Linux 4b7cd67c0132 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / 0ccf440
Max. process+thread count 71 (vs. ulimit of 5500)
modules C: docs U: docs
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-148/1/console
versions git=2.25.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@jteagles jteagles merged commit c875b82 into apache:master Aug 26, 2021
@jteagles
Copy link
Contributor

That logo is pretty big. Scale it down?

@abstractdog
Copy link
Contributor Author

That logo is pretty big. Scale it down?

yeah, I'll take care of it (forgot to double-check in browser before committing to svn repo)

@abstractdog
Copy link
Contributor Author

UPDATE: reverted from svn until I can do the proper design locally

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