Skip to content

PHOENIX-6480 Move phoenix-tool and add support with generating defaul… - #1279

Merged
richardantal merged 1 commit into
apache:masterfrom
richardantal:PHOENIX-6480
Aug 2, 2021
Merged

PHOENIX-6480 Move phoenix-tool and add support with generating defaul…#1279
richardantal merged 1 commit into
apache:masterfrom
richardantal:PHOENIX-6480

Conversation

@richardantal

Copy link
Copy Markdown
Contributor

…t properties

…t properties (apache#1245)
Change-Id: I17241f3f93dbdab08be3c151404d19dd6bd78626
@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+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 🆗mvndep5m 0sMaven dependency ordering for branch
+1 💚mvninstall17m 9smaster passed
+0hbaserecompile28m 47sHBase recompiled.
+1 💚compile1m 51smaster passed
+1 💚checkstyle1m 14smaster passed
+1 💚javadoc2m 58smaster passed
+0 🆗spotbugs4m 54sroot in master has 1032 extant spotbugs warnings.
+0 🆗spotbugs3m 5sphoenix-core in master has 970 extant spotbugs warnings.
+0 🆗spotbugs0m 39sphoenix-tools in master has 3 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall14m 6sthe patch passed
+0hbaserecompile20m 28sHBase recompiled.
+1 💚compile1m 40sthe patch passed
+1 💚javac1m 40sthe patch passed
-1 ❌checkstyle1m 6sroot: The patch generated 27 new + 317 unchanged - 25 fixed = 344 total (was 342)
-1 ❌whitespace0m 0sThe patch 2 line(s) with tabs.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc2m 21sthe patch passed
-1 ❌spotbugs3m 29sphoenix-core generated 3 new + 970 unchanged - 0 fixed = 973 total (was 970)
_ Other Tests _
-1 ❌unit124m 25sroot in the patch failed.
-1 ❌asflicense2m 24sThe patch generated 614 ASF License warnings.
209m 25s
ReasonTests
FindBugsmodule:phoenix-core
org.apache.phoenix.schema.tool.SchemaTool.conf should be package protected In SchemaTool.java: In SchemaTool.java
Possible null pointer dereference of cmdLine in org.apache.phoenix.schema.tool.SchemaTool.parseOptions(String[]) Dereferenced at SchemaTool.java:cmdLine in org.apache.phoenix.schema.tool.SchemaTool.parseOptions(String[]) Dereferenced at SchemaTool.java:[line 114]
Write to static field org.apache.phoenix.schema.tool.SchemaTool.conf from instance method org.apache.phoenix.schema.tool.SchemaTool.run(String[]) At SchemaTool.java:from instance method org.apache.phoenix.schema.tool.SchemaTool.run(String[]) At SchemaTool.java:[line 73]
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1279
Optional Testsdupname asflicense javac javadoc unit xml compile spotbugs hbaserebuild hbaseanti checkstyle
unameLinux 5ebc2ecf8639 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 / a4f0304
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/diff-checkstyle-root.txt
whitespacehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/whitespace-tabs.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count9680 (vs. ulimit of 30000)
modulesC: phoenix-core phoenix-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1279/1/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@richardantal
richardantal merged commit 025a4af into apache:masterAug 2, 2021
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

@richardantal@stoty@gokceni