Skip to content

PHOENIX-6186 - Store last DDL timestamp in System.Catalog (5.x) - #977

Merged
gjacoby126 merged 1 commit into
apache:masterfrom
gjacoby126:PHOENIX-6186-master
Nov 20, 2020
Merged

PHOENIX-6186 - Store last DDL timestamp in System.Catalog (5.x)#977
gjacoby126 merged 1 commit into
apache:masterfrom
gjacoby126:PHOENIX-6186-master

Conversation

@gjacoby126

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 42sDocker 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 appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep3m 38sMaven dependency ordering for branch
+1 💚mvninstall11m 42smaster passed
+1 💚compile1m 26smaster passed
+1 💚checkstyle7m 30smaster passed
+1 💚javadoc1m 58smaster passed
+0 🆗spotbugs3m 7sphoenix-core in master has 965 extant spotbugs warnings.
+0 🆗spotbugs4m 11sroot in master has 1014 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for patch
+1 💚mvninstall9m 26sthe patch passed
+1 💚compile1m 25sthe patch passed
+1 💚cc1m 25sthe patch passed
+1 💚javac1m 25sthe patch passed
-1 ❌checkstyle7m 26sroot: The patch generated 371 new + 25007 unchanged - 255 fixed = 25378 total (was 25262)
+1 💚prototool0m 2sThere were no new prototool issues.
-1 ❌whitespace0m 0sThe patch 1 line(s) with tabs.
-1 ❌javadoc0m 51sphoenix-core generated 1 new + 99 unchanged - 1 fixed = 100 total (was 100)
-1 ❌javadoc1m 13sroot generated 1 new + 131 unchanged - 1 fixed = 132 total (was 132)
-1 ❌spotbugs3m 40sphoenix-core generated 2 new + 964 unchanged - 1 fixed = 966 total (was 965)
-1 ❌spotbugs4m 47sroot generated 2 new + 1013 unchanged - 1 fixed = 1015 total (was 1014)
_ Other Tests _
-1 ❌unit168m 43sroot in the patch failed.
+1 💚asflicense0m 44sThe patch does not generate ASF License warnings.
238m 1s
ReasonTests
FindBugsmodule:phoenix-core
org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.LAST_DDL_TIMESTAMP_BYTES is a mutable array At MetaDataEndpointImpl.java: At MetaDataEndpointImpl.java:[line 337]
A prepared statement is generated from a nonconstant String in org.apache.phoenix.util.UpgradeUtil.bootstrapLastDDLTimestamp(Connection) At UpgradeUtil.java:from a nonconstant String in org.apache.phoenix.util.UpgradeUtil.bootstrapLastDDLTimestamp(Connection) At UpgradeUtil.java:[line 2616]
FindBugsmodule:root
org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.LAST_DDL_TIMESTAMP_BYTES is a mutable array At MetaDataEndpointImpl.java: At MetaDataEndpointImpl.java:[line 337]
A prepared statement is generated from a nonconstant String in org.apache.phoenix.util.UpgradeUtil.bootstrapLastDDLTimestamp(Connection) At UpgradeUtil.java:from a nonconstant String in org.apache.phoenix.util.UpgradeUtil.bootstrapLastDDLTimestamp(Connection) At UpgradeUtil.java:[line 2616]
SubsystemReport/Notes
DockerClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#977
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile cc prototool
unameLinux 104e3966bebc 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 / 2af2adf
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-977/4/artifact/yetus-general-check/output/diff-checkstyle-root.txt
whitespacehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/whitespace-tabs.txt
javadochttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix-core.txt
javadochttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/diff-javadoc-javadoc-root.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/new-spotbugs-root.html
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/testReport/
Max. process+thread count6271 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-977/4/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3 prototool=1.10.0-dev
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@gjacoby126

Copy link
Copy Markdown
ContributorAuthor

Tests passed; the failure seems to come from jacoco crashing afterward. Since this is just a port of the previously approved 4.x version of PHOENIX-6186, I'm going to go ahead and merge.

@gjacoby126
gjacoby126 merged commit f1a0860 into apache:masterNov 20, 2020
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

@gjacoby126@stoty