Skip to content

PHOENIX-6551 Bump HBase version to 2.4.6 and 2.2.7 - #1313

Closed
stoty wants to merge 1 commit into
apache:masterfrom
stoty:PHOENIX-6551
Closed

PHOENIX-6551 Bump HBase version to 2.4.6 and 2.2.7#1313
stoty wants to merge 1 commit into
apache:masterfrom
stoty:PHOENIX-6551

Conversation

@stoty

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec8m 21sDocker 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 💚mvninstall30m 28smaster passed
+0hbaserecompile39m 38sHBase recompiled.
+1 💚compile2m 35smaster passed
+1 💚javadoc2m 1smaster passed
_ Patch Compile Tests _
-1 ❌mvninstall0m 46sroot in the patch failed.
+0hbaserecompile55m 45sHBase recompiled.
+1 💚compile2m 27sthe patch passed
+1 💚javac2m 27sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc2m 3sthe patch passed
_ Other Tests _
-1 ❌unit247m 26sroot in the patch failed.
-1 ❌asflicense1m 9sThe patch generated 612 ASF License warnings.
319m 56s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1313
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux f05dbdbc355d 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 8fb38e4
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/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-1313/1/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count6862 (vs. ulimit of 30000)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/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.

@virajjasani

Copy link
Copy Markdown
Contributor

Looks like some issue resolving artifacts, let me recheck repositories content and retrigger the build if everything is fine:

The following artifacts could not be resolved: org.apache.hbase:hbase-common:jar:2.4.6, org.apache.hbase:hbase-mapreduce:jar:2.4.6, org.apache.hbase:hbase-metrics-api:jar:2.4.6, org.apache.hbase:hbase-metrics:jar:2.4.6, org.apache.hbase:hbase-protocol:jar:2.4.6, org.apache.hbase:hbase-protocol-shaded:jar:2.4.6, org.apache.hbase:hbase-client:jar:2.4.6, org.apache.hbase:hbase-server:jar:2.4.6, org.apache.hbase:hbase-hadoop-compat:jar:2.4.6, org.apache.hbase:hbase-hadoop2-compat:jar:2.4.6, org.apache.hbase:hbase-zookeeper:jar:2.4.6, org.apache.hbase:hbase-it:jar:tests:2.4.6, org.apache.hbase:hbase-common:jar:tests:2.4.6, org.apache.hbase:hbase-server:jar:tests:2.4.6, org.apache.hbase:hbase-testing-util:jar:2.4.6, org.apache.hbase:hbase-endpoint:jar:2.4.6: Could not find artifact org.apache.hbase:hbase-common:jar:2.4.6 in apache release (https://repository.apache.org/content/repositories/releases/) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

@stoty

stoty commented Sep 20, 2021

Copy link
Copy Markdown
ContributorAuthor

Interesting.
The CI checks recompile hbase from the officially released source to get around the Hadoop3 issue, and so did I in my local tests.
It seems that the 2.4.6 release process hasn't been completed, and the artifacts weren't pulished to maven.
We should probably wait until that is resolved. (and notify the HBase project)

@stoty

Copy link
Copy Markdown
ContributorAuthor

@apurtell Do you know the reason ?

@virajjasani

Copy link
Copy Markdown
Contributor

@stoty It's resolved now, just triggered the QA run.

@stoty

Copy link
Copy Markdown
ContributorAuthor

I still cannot see 2.4.6 on https://repo.maven.apache.org/maven2/org/apache/hbase/hbase-client/
(though it may just take some time to catch up)

@virajjasani

Copy link
Copy Markdown
Contributor

I still cannot see 2.4.6 on https://repo.maven.apache.org/maven2/org/apache/hbase/hbase-client/
(though it may just take some time to catch up)

Yeah it should show up after some time. It's present here in r.a.o though https://repository.apache.org/content/repositories/releases/org/apache/hbase/hbase-client/2.4.6/

@stoty

Copy link
Copy Markdown
ContributorAuthor

It has appeared. Thanks for your intervention.
However, we do not use the public hbase maven artifacts anywhere in the CI pipeline for 5.x, so it shouldn't make a difference for the CI run.

@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 14sDocker 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 💚mvninstall25m 1smaster passed
+0hbaserecompile32m 46sHBase recompiled.
+1 💚compile2m 0smaster passed
+1 💚javadoc1m 27smaster passed
_ Patch Compile Tests _
+1 💚mvninstall18m 38sthe patch passed
+0hbaserecompile62m 39sHBase recompiled.
+1 💚compile1m 52sthe patch passed
+1 💚javac1m 52sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc1m 18sthe patch passed
_ Other Tests _
-1 ❌unit126m 26sroot in the patch failed.
-1 ❌asflicense1m 6sThe patch generated 612 ASF License warnings.
197m 3s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1313
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux ae52bababa32 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / f606289
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-1313/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-1313/2/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count11365 (vs. ulimit of 30000)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1313/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.

@virajjasanivirajjasani 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, thank you @stoty

@stotystoty closed this Sep 20, 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

@stoty@virajjasani