Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRILL-8515: Upgrade to HBase client 2.6.1 #2968

Merged
merged 2 commits into from
Jan 2, 2025

Conversation

pjfanning
Copy link
Contributor

@pjfanning pjfanning commented Dec 26, 2024

Description

DRILL-8515

Upgrade HBase client due to issues with zookeeper upgrade - #2893

Documentation

(Please describe user-visible changes similar to what should appear in the Drill documentation.)

Testing

(Please describe how this PR has been tested.)

@pjfanning
Copy link
Contributor Author

pjfanning commented Dec 26, 2024

Running into (storage-phoenix plugin)

Caused by: java.lang.NoSuchMethodError: 'io.opentelemetry.api.trace.Span org.apache.hadoop.hbase.trace.TraceUtil.createSpan(java.lang.String)'
	at org.apache.hadoop.hbase.regionserver.HRegionServer.<init>(HRegionServer.java:631)
	at org.apache.hadoop.hbase.master.HMaster.<init>(HMaster.java:474)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490)
	at org.apache.hadoop.hbase.util.JVMClusterUtil.createMasterThread(JVMClusterUtil.java:124)

@cgivre
Copy link
Contributor

cgivre commented Jan 1, 2025

@pjfanning Could you please rebase on current master? I'll rerun the CI once you've done that to see if we can get a clean run.

@pjfanning
Copy link
Contributor Author

The issue in #2968 (comment) is still happening even after a rebase. I haven't yet had time to investigate why.

Copy link
Contributor

@cgivre cgivre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

Thanks @pjfanning

@cgivre cgivre changed the title [DRILL-8515] upgrade to hbase client 2.6.1 DRILL-8515: Upgrade to HBase client 2.6.1 Jan 2, 2025
@cgivre cgivre merged commit 1ed8253 into apache:master Jan 2, 2025
8 checks passed
@pjfanning pjfanning deleted the DRILL-8515-hbase branch January 2, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants