Skip to content

PHOENIX-6446 : Bump default HBase version on 2.3 profile to 2.3.5 - #1199

Merged
virajjasani merged 3 commits into
apache:masterfrom
virajjasani:PHOENIX-6446-master
Apr 19, 2021
Merged

PHOENIX-6446 : Bump default HBase version on 2.3 profile to 2.3.5#1199
virajjasani merged 3 commits into
apache:masterfrom
virajjasani:PHOENIX-6446-master

Conversation

@virajjasani

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 35sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+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 _
+1 💚mvninstall20m 23smaster passed
+0hbaserecompile29m 1sHBase recompiled.
+1 💚compile1m 46smaster passed
+1 💚javadoc1m 45smaster passed
_ Patch Compile Tests _
+1 💚mvninstall11m 50sthe patch passed
+0hbaserecompile53m 20sHBase recompiled.
+1 💚compile2m 2sthe patch passed
+1 💚javac2m 2sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc1m 43sthe patch passed
_ Other Tests _
-1 ❌unit139m 2sroot in the patch failed.
-1 ❌asflicense1m 8sThe patch generated 613 ASF License warnings.
201m 27s
ReasonTests
Failed junit testsphoenix.end2end.PermissionNSEnabledWithCustomAccessControllerIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1199
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux 8e9448039348 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 84d317c
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/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-1199/1/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count6993 (vs. ulimit of 30000)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/1/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+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 19sMaven dependency ordering for branch
+1 💚mvninstall14m 1smaster passed
+0hbaserecompile26m 21sHBase recompiled.
+1 💚compile1m 33smaster passed
+1 💚javadoc2m 31smaster passed
_ Patch Compile Tests _
+0 🆗mvndep1m 6sMaven dependency ordering for patch
+1 💚mvninstall10m 18sthe patch passed
+0hbaserecompile18m 15sHBase recompiled.
+1 💚compile1m 37sthe patch passed
+1 💚javac1m 37sthe patch passed
+1 💚whitespace0m 1sThe patch has no whitespace issues.
+1 💚xml0m 4sThe patch has no ill-formed XML file.
+1 💚javadoc2m 30sthe patch passed
_ Other Tests _
-1 ❌unit120m 56sroot in the patch failed.
-1 ❌asflicense1m 53sThe patch generated 620 ASF License warnings.
180m 19s
ReasonTests
Failed junit testsphoenix.end2end.MigrateSystemTablesToSystemNamespaceIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1199
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux 9dcfad4f79c1 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 84d317c
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/2/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/2/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count10101 (vs. ulimit of 30000)
modulesC: phoenix-core phoenix-server . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/2/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@virajjasani

Copy link
Copy Markdown
ContributorAuthor

Reverted the commit that was meant to run tests against 2.3.5

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 4sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+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 17sMaven dependency ordering for branch
+1 💚mvninstall12m 47smaster passed
+0hbaserecompile25m 10sHBase recompiled.
+1 💚compile1m 34smaster passed
+1 💚javadoc2m 33smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall10m 8sthe patch passed
+0hbaserecompile17m 35sHBase recompiled.
+1 💚compile1m 37sthe patch passed
+1 💚javac1m 37sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚javadoc2m 30sthe patch passed
_ Other Tests _
-1 ❌unit118m 5sroot in the patch failed.
-1 ❌asflicense1m 52sThe patch generated 613 ASF License warnings.
175m 16s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1199
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux 130420250ffa 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 84d317c
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/3/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/3/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/3/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count12325 (vs. ulimit of 30000)
modulesC: phoenix-core phoenix-server . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1199/3/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@virajjasani
virajjasani requested a review from stotyApril 18, 2021 11:15

@stotystoty left a comment

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.

+1 LGTM

@virajjasani
virajjasani merged commit f4ee13b into apache:masterApr 19, 2021
@virajjasani
virajjasani deleted the PHOENIX-6446-master branch April 19, 2021 09:10
virajjasani added a commit that referenced this pull request Apr 19, 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.

2 participants

@virajjasani@stoty