Skip to content

PHOENIX-6114 Create shaded phoenix-pherf and remove lib dir from asse… - #1131

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

PHOENIX-6114 Create shaded phoenix-pherf and remove lib dir from asse…#1131
stoty wants to merge 1 commit into
apache:masterfrom
stoty:PHOENIX-6114

Conversation

@stoty

@stotystoty commented Feb 4, 2021

Copy link
Copy Markdown
Contributor

…mbly

remove pherf-cluster.py
remove phoenix-pherf-minimal shaded jar
remove /lib dir from assembly
remove /dev library from assemby
shade phoenix-pherf instead
move phoenix-pherf.jar to root
update scripts for new jar names and paths

@stoty

stoty commented Feb 4, 2021

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 _
+0 🆗mvndep4m 58sMaven dependency ordering for branch
+1 💚mvninstall13m 38smaster passed
+0hbaserecompile27m 13sHBase recompiled.
+1 💚compile1m 57smaster passed
+1 💚javadoc2m 29smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
-1 ❌mvninstall10m 57sroot in the patch failed.
+0hbaserecompile19m 50sHBase recompiled.
+1 💚compile2m 21sthe patch passed
+1 💚javac2m 21sthe patch passed
+1 💚pylint0m 7sThe patch generated 0 new + 87 unchanged - 31 fixed = 87 total (was 118)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 6sThe patch has no ill-formed XML file.
+1 💚javadoc2m 3sthe patch passed
_ Other Tests _
-1 ❌unit129m 17sroot in the patch failed.
-1 ❌asflicense1m 2sThe patch generated 612 ASF License warnings.
188m 33s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1131
Optional Testsdupname asflicense pylint xml javac javadoc unit compile
unameLinux b1df78f3e909 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / d2be16d
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-1131/2/artifact/yetus-general-check/output/patch-mvninstall-root.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/2/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/2/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count13574 (vs. ulimit of 30000)
modulesC: phoenix-pherf phoenix-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/2/console
versionsgit=2.7.4 maven=3.3.9 pylint=2.4.4
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@yanxinyi

Copy link
Copy Markdown
Contributor

My local build and I precommit build failed because of the same reason. This is only for the master branch but not 4.x

./patch-mvninstall-root.txt:[ERROR] Failed to execute goal org.apache.maven.plugins:maven-install-plugin:2.5.2:install-file (default-install) on project phoenix-pherf: The specified file '/home/jenkins/jenkins-home/workspace/enix-PreCommit-GitHub-PR_PR-1131/yetus-general-check/src/phoenix-pherf/target/phoenix-pherf-5.1.0-SNAPSHOT-minimal.jar' not exists -> [Help 1]

@yanxinyi

Copy link
Copy Markdown
Contributor

Ok. I guess you probably forgot to remove this
https://github.com/apache/phoenix/blob/master/phoenix-pherf/pom.xml#L299

…mbly
remove pherf-cluster.py
remove phoenix-pherf-minimal shaded jar
remove /lib dir from assembly
remove /dev library from assemby
shade phoenix-pherf instead
move phoenix-pherf.jar to root
update scripts for new jar names and paths
@stoty

stoty commented Feb 5, 2021

Copy link
Copy Markdown
ContributorAuthor

Thank you @yanxinyi .
Yes. I've fixed that on the 4.x PR, but not here.
Pushed the fix now.

@yanxinyiyanxinyi 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 LGTM

@stotystoty closed this Feb 5, 2021
@stoty

stoty commented Feb 5, 2021

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec5m 42sDocker 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 _
+0 🆗mvndep4m 55sMaven dependency ordering for branch
+1 💚mvninstall15m 19smaster passed
+0hbaserecompile28m 51sHBase recompiled.
+1 💚compile2m 2smaster passed
+1 💚javadoc2m 34smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall11m 52sthe patch passed
+0hbaserecompile20m 29sHBase recompiled.
+1 💚compile2m 9sthe patch passed
+1 💚javac2m 9sthe patch passed
+1 💚pylint0m 9sThe patch generated 0 new + 87 unchanged - 31 fixed = 87 total (was 118)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 7sThe patch has no ill-formed XML file.
+1 💚javadoc2m 13sthe patch passed
_ Other Tests _
-1 ❌unit104m 21sroot in the patch failed.
-1 ❌asflicense2m 24sThe patch generated 612 ASF License warnings.
174m 36s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1131
Optional Testsdupname asflicense pylint xml javac javadoc unit compile
unameLinux 3492faedc572 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / d2be16d
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/3/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/3/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/3/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count13390 (vs. ulimit of 30000)
modulesC: phoenix-pherf phoenix-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1131/3/console
versionsgit=2.7.4 maven=3.3.9 pylint=2.4.4
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty
stoty deleted the PHOENIX-6114 branch July 29, 2021 12:06
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@yanxinyi