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

HBase Connection Registry usage with Phoenix JDBC url #24869

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

virajjasani
Copy link
Member

Description

Use various HBase Connection Registries with Phoenix JDBC url formats. Document with recommendations and use in test.

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)

@virajjasani
Copy link
Member Author

virajjasani commented Jan 31, 2025

I have run TestPhoenixConnectorTest locally 3 times and all 241 out of 337 (96 ignored) tests are passing each time. Not sure why the registry based exception would occur (unless the master daemon is not stable or does not have meta cache).

Edit: updated test to use new format for ZK ConnectionRegistry. RPC Registry for test is facing errors because in docker env, clients can exhaust master with RPC threadpool while it is already running in-house chores.

@virajjasani virajjasani force-pushed the reg branch 4 times, most recently from d19451c to 77e2c2c Compare February 1, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant