Skip to content

PHOENIX-6399 Updating BackwardCompatibilityIT supported versions - #1164

Closed
stoty wants to merge 1 commit into
apache:4.xfrom
stoty:PHOENIX-6399
Closed

PHOENIX-6399 Updating BackwardCompatibilityIT supported versions#1164
stoty wants to merge 1 commit into
apache:4.xfrom
stoty:PHOENIX-6399

Conversation

@stoty

Copy link
Copy Markdown
Contributor

No description provided.

@stoty
stoty requested a review from yanxinyiMarch 10, 2021 08:19
@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 9sDocker 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.
_ 4.x Compile Tests _
+1 💚mvninstall15m 7s4.x passed
+1 💚compile1m 5s4.x passed
+1 💚checkstyle0m 27s4.x passed
+1 💚javadoc0m 49s4.x passed
+0 🆗spotbugs3m 16sphoenix-core in 4.x has 944 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall6m 49sthe patch passed
+1 💚compile1m 6sthe patch passed
+1 💚javac1m 6sthe patch passed
+1 💚checkstyle0m 28sthe patch passed
-1 ❌shellcheck0m 0sThe patch generated 6 new + 6 unchanged - 2 fixed = 12 total (was 8)
+1 💚shelldocs0m 2sThere were no new shelldocs issues.
-1 ❌whitespace0m 0sThe patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚javadoc0m 47sthe patch passed
+1 💚spotbugs3m 25sthe patch passed
_ Other Tests _
+1 💚unit197m 15sphoenix-core in the patch passed.
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
232m 51s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1164/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1164
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile shellcheck shelldocs
unameLinux e8f75cd75ecc 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / 78a5ab0
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
shellcheckhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1164/1/artifact/yetus-general-check/output/diff-patch-shellcheck.txt
whitespacehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1164/1/artifact/yetus-general-check/output/whitespace-eol.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1164/1/testReport/
Max. process+thread count4850 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1164/1/console
versionsgit=2.7.4 maven=3.3.9 shellcheck=0.7.0 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

"1.4": [ {"artifactId":"phoenix-client", "version":"4.14.3-HBase-1.4"}, {"artifactId":"phoenix-client", "version":"4.15.0-HBase-1.4"}, {"artifactId":"phoenix-client-hbase-1.4", "version":"4.16.0"} ],
"1.5": [ {"artifactId":"phoenix-client", "version":"4.15-HBase-1.5"}, {"artifactId":"phoenix-client-hbase-1.5", "version":"4.16.0"} ],
"1.6": [ {"artifactId":"phoenix-client-hbase-1.6", "version":"4.16.0"} ],
"2.1": [ {"artifactId":"phoenix-client-hbase-2.1", "version":"5.1.0"} ],

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.

nit: I guess we don't need 2.x here, but it won't run so it shouldn't matter

@stotystoty closed this Mar 11, 2021
@stoty
stoty deleted the PHOENIX-6399 branch July 29, 2021 14:38
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