Skip to content

feat: expose Hive client pools - #155

Merged
geruh merged 3 commits into
lance-format:mainfrom
hfutatzhanghb:agent/expose-hive-client-pools
Oct 5, 2026
Merged

geruh merged 3 commits into
lance-format:mainfrom
hfutatzhanghb:agent/expose-hive-client-pools

Conversation

@hfutatzhanghb

@hfutatzhanghb hfutatzhanghb commented Jul 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • expose the namespace-owned Hive2ClientPool from Hive2Namespace
  • expose the namespace-owned Hive3ClientPool from Hive3Namespace
  • add API tests that verify each getter returns the initialized metastore pool

Why

Downstream integrations such as lance-spark need to perform metastore work through the same configured client pool as the namespace.

Validation

  • Added Hive 2 and Hive 3 API tests.

@github-actions github-actions Bot added enhancement New feature or request java labels Jul 10, 2026
@hfutatzhanghb
hfutatzhanghb marked this pull request as ready for review July 10, 2026 09:08
@hfutatzhanghb

Copy link
Copy Markdown
Contributor Author

Hi, @puchengy . Could you please help review and merge this PR when free ? Thanks a lot !

@puchengy

Copy link
Copy Markdown
Contributor

@hfutatzhanghb Hey, thanks for the work but unfortunately I don't have a merge permission. Sorry!

@hfutatzhanghb

Copy link
Copy Markdown
Contributor Author

@hfutatzhanghb Hey, thanks for the work but unfortunately I don't have a merge permission. Sorry!

Got it. Thanks!

@hfutatzhanghb

Copy link
Copy Markdown
Contributor Author

Hi, @jackye1995 Could you please help review and merge this PR when free ? Thanks a lot !

@geruh
geruh merged commit 72268e5 into lance-format:main Oct 5, 2026
10 checks passed
@hfutatzhanghb
hfutatzhanghb deleted the agent/expose-hive-client-pools branch October 8, 2026 02:01
@hfutatzhanghb

Copy link
Copy Markdown
Contributor Author

Thanks very much! @geruh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants