phoenix icon indicating copy to clipboard operation
phoenix copied to clipboard

PHOENIX-3165 PHCK tool

Open yanxinyi opened this issue 5 years ago • 1 comments

This is the draft version of the phck tool PR. Since this PR most likely will be a large PR, it's better to review the first part of the feature, which contains the main phck framework, as well as system-level table monitor/fix mode.

Design doc: https://docs.google.com/document/d/103xknWmz2EqQ7LqqMeb4cfJ_5pwuRbAZFMdVVJoiNCw/edit

yanxinyi avatar May 29 '20 01:05 yanxinyi

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 5m 22s Docker mode activated.
_ Prechecks _
+1 :green_heart: dupname 0m 0s No case conflicting files found.
+1 :green_heart: hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 :green_heart: @author 0m 0s The patch does not contain any @author tags.
+1 :green_heart: test4tests 0m 0s The patch appears to include 2 new or modified test files.
_ 4.x Compile Tests _
+1 :green_heart: mvninstall 14m 35s 4.x passed
+1 :green_heart: compile 1m 1s 4.x passed
+1 :green_heart: checkstyle 0m 28s 4.x passed
+1 :green_heart: javadoc 0m 44s 4.x passed
+0 :ok: spotbugs 2m 58s phoenix-core in 4.x has 945 extant spotbugs warnings.
_ Patch Compile Tests _
-1 :x: mvninstall 0m 58s root in the patch failed.
-1 :x: compile 0m 51s phoenix-core in the patch failed.
-1 :x: javac 0m 51s phoenix-core in the patch failed.
-1 :x: checkstyle 0m 37s phoenix-core: The patch generated 361 new + 0 unchanged - 0 fixed = 361 total (was 0)
+1 :green_heart: whitespace 0m 0s The patch has no whitespace issues.
+1 :green_heart: javadoc 0m 43s the patch passed
-1 :x: spotbugs 0m 48s phoenix-core in the patch failed.
_ Other Tests _
-1 :x: unit 0m 50s phoenix-core in the patch failed.
+1 :green_heart: asflicense 0m 10s The patch does not generate ASF License warnings.
30m 53s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR https://github.com/apache/phoenix/pull/791
Optional Tests dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
uname Linux 92a17e7fc5e9 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 tool maven
Personality dev/phoenix-personality.sh
git revision 4.x / ec386e9
Default Java Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
mvninstall https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt
compile https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt
javac https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt
checkstyle https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
spotbugs https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/patch-spotbugs-phoenix-core.txt
unit https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt
Test Results https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/testReport/
Max. process+thread count 94 (vs. ulimit of 30000)
modules C: phoenix-core U: phoenix-core
Console output https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-791/1/console
versions git=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

stoty avatar Feb 16 '21 03:02 stoty