Skip to content

TEZ-4533: Source release distribution must not contain TestIFile_concatenated_compressed.bin - #328

Closed
zabetak wants to merge 1 commit into
apache:masterfrom
zabetak:TEZ-4533
Closed

TEZ-4533: Source release distribution must not contain TestIFile_concatenated_compressed.bin#328
zabetak wants to merge 1 commit into
apache:masterfrom
zabetak:TEZ-4533

Conversation

@zabetak

Copy link
Copy Markdown
Member

TestIFile_concatenated_compressed.bin file must not be present in the source distribution. To avoid test failures when building from source just skip the test if the file is missing.

Since the creation of the src.tar.gz archive is manual the respective instructions have to be update to exclude explicitly this file:
https://cwiki.apache.org/confluence/display/TEZ/Making+a+TEZ+Release#MakingaTEZRelease-CreateaReleasetarballfortheSource

…atenated_compressed.bin
TestIFile_concatenated_compressed.bin file must not be present in the source distribution. To avoid test failures when building from source just skip the test if the file is missing.
@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec25m 12sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 💚mvninstall17m 35smaster passed
+1 💚compile0m 35smaster passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu122.04
+1 💚compile0m 32smaster passed with JDK Private Build-1.8.0_392-8u392-ga-1~22.04-b08
+1 💚checkstyle1m 23smaster passed
+1 💚javadoc0m 44smaster passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu122.04
+1 💚javadoc0m 30smaster passed with JDK Private Build-1.8.0_392-8u392-ga-1~22.04-b08
+0 🆗spotbugs1m 34sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs1m 32smaster passed
_ Patch Compile Tests _
+1 💚mvninstall0m 21sthe patch passed
+1 💚compile0m 23sthe patch passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu122.04
+1 💚javac0m 23sthe patch passed
+1 💚compile0m 20sthe patch passed with JDK Private Build-1.8.0_392-8u392-ga-1~22.04-b08
+1 💚javac0m 20sthe patch passed
-0 ⚠️checkstyle0m 15stez-runtime-library: The patch generated 1 new + 17 unchanged - 0 fixed = 18 total (was 17)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 19sthe patch passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu122.04
+1 💚javadoc0m 18sthe patch passed with JDK Private Build-1.8.0_392-8u392-ga-1~22.04-b08
+1 💚findbugs0m 57sthe patch passed
_ Other Tests _
+1 💚unit5m 27stez-runtime-library in the patch passed.
+1 💚asflicense0m 16sThe patch does not generate ASF License warnings.
57m 32s
SubsystemReport/Notes
DockerClientAPI=1.44 ServerAPI=1.44 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-328/1/artifact/out/Dockerfile
GITHUB PR#328
JIRA IssueTEZ-4533
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 57753f8c332f 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 2161124
Default JavaPrivate Build-1.8.0_392-8u392-ga-1~22.04-b08
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_392-8u392-ga-1~22.04-b08
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-328/1/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-328/1/testReport/
Max. process+thread count1090 (vs. ulimit of 5500)
modulesC: tez-runtime-library U: tez-runtime-library
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-328/1/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor

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

@zabetak@tez-yetus@abstractdog