Skip to content

PHOENIX-6495 Include phoenix-tools jar in assembly - #1253

Closed
stoty wants to merge 1 commit into
apache:masterfrom
stoty:PHOENIX-6495
Closed

PHOENIX-6495 Include phoenix-tools jar in assembly#1253
stoty wants to merge 1 commit into
apache:masterfrom
stoty:PHOENIX-6495

Conversation

@stoty

Copy link
Copy Markdown
Contributor

also restore/add versionsless symlinks

@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec4m 15sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
-1 ❌mvninstall15m 35sroot in master failed.
+0hbaserecompile23m 38sHBase recompiled.
-1 ❌compile0m 14sphoenix-assembly in master failed.
-1 ❌javadoc0m 14sphoenix-assembly in master failed.
_ Patch Compile Tests _
-1 ❌mvninstall7m 55sroot in the patch failed.
+0hbaserecompile40m 11sHBase recompiled.
-1 ❌compile0m 14sphoenix-assembly in the patch failed.
-1 ❌javac0m 14sphoenix-assembly in the patch failed.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
-1 ❌javadoc0m 14sphoenix-assembly in the patch failed.
_ Other Tests _
-1 ❌unit0m 14sphoenix-assembly in the patch failed.
+1 💚asflicense0m 15sThe patch does not generate ASF License warnings.
46m 10s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1253
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux 7ed7bed63c1e 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 8b81579
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/branch-mvninstall-root.txt
compilehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/branch-compile-phoenix-assembly.txt
javadochttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/branch-javadoc-phoenix-assembly.txt
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt
compilehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/patch-compile-phoenix-assembly.txt
javachttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/patch-compile-phoenix-assembly.txt
javadochttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/patch-javadoc-phoenix-assembly.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/artifact/yetus-general-check/output/patch-unit-phoenix-assembly.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/testReport/
Max. process+thread count96 (vs. ulimit of 30000)
modulesC: phoenix-assembly U: phoenix-assembly
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/1/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

also restore/add versionsless symlinks
@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 32sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1 💚mvninstall26m 0smaster passed
+0hbaserecompile33m 52sHBase recompiled.
+1 💚compile0m 29smaster passed
+1 💚javadoc0m 17smaster passed
_ Patch Compile Tests _
+1 💚mvninstall16m 47sthe patch passed
+0hbaserecompile59m 54sHBase recompiled.
+1 💚compile0m 35sthe patch passed
+1 💚javac0m 35sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚javadoc0m 17sthe patch passed
_ Other Tests _
+1 💚unit1m 4sphoenix-assembly in the patch passed.
+1 💚asflicense0m 16sThe patch does not generate ASF License warnings.
63m 28s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1253
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux f83792eca1df 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / f9070c3
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/2/testReport/
Max. process+thread count96 (vs. ulimit of 30000)
modulesC: phoenix-assembly U: phoenix-assembly
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1253/2/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stotystoty closed this Jun 19, 2021
@stoty
stoty deleted the PHOENIX-6495 branch July 29, 2021 14:42
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.

2 participants

@stoty@virajjasani