PHOENIX-6271: Effective DDL generated by SchemaExtractionTool should …

[stoty]

…maintain the order of PK and other columns (#1218)

[stoty]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	4m 25s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 0s	No case conflicting files found.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
-1 ❌	test4tests	0m 0s	The 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.
		_ 5.1 Compile Tests _
+0 🆗	mvndep	5m 56s	Maven dependency ordering for branch
+1 💚	mvninstall	10m 51s	5.1 passed
+0	hbaserecompile	24m 38s	HBase recompiled.
+1 💚	compile	1m 34s	5.1 passed
+1 💚	checkstyle	1m 8s	5.1 passed
+1 💚	javadoc	1m 12s	5.1 passed
+0 🆗	spotbugs	3m 5s	phoenix-core in 5.1 has 954 extant spotbugs warnings.
+0 🆗	spotbugs	0m 42s	phoenix-tools in 5.1 has 3 extant spotbugs warnings.
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 12s	Maven dependency ordering for patch
+1 💚	mvninstall	8m 1s	the patch passed
+0	hbaserecompile	16m 24s	HBase recompiled.
+1 💚	compile	1m 35s	the patch passed
+1 💚	javac	1m 35s	the patch passed
-1 ❌	checkstyle	0m 50s	phoenix-core: The patch generated 2 new + 702 unchanged - 3 fixed = 704 total (was 705)
-1 ❌	checkstyle	0m 21s	phoenix-tools: The patch generated 11 new + 215 unchanged - 4 fixed = 226 total (was 219)
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 1s	The patch has no ill-formed XML file.
+1 💚	javadoc	1m 9s	the patch passed
+1 💚	spotbugs	4m 10s	the patch passed
		_ Other Tests _
-1 ❌	unit	108m 31s	phoenix-core in the patch failed.
+1 💚	unit	3m 56s	phoenix-tools in the patch passed.
+1 💚	asflicense	1m 12s	The patch does not generate ASF License warnings.
		178m 32s

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-1251/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#1251
Optional Tests	dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
uname	Linux 69a8c3a73210 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 tool	maven
Personality	dev/phoenix-personality.sh
git revision	5.1 / e6d8f40
Default Java	Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
checkstyle	https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1251/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
checkstyle	https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1251/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-tools.txt
unit	https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1251/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-1251/1/testReport/
Max. process+thread count	13996 (vs. ulimit of 30000)
modules	C: phoenix-core phoenix-tools U: .
Console output	https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1251/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.