Skip to content

Use RouterBasedStoreMetadataFetcher for Venice store discovery#228

Open
srnand wants to merge 2 commits into
mainfrom
srnand/may20_26/UseVeniceStoreMetadataFetcher
Open

Use RouterBasedStoreMetadataFetcher for Venice store discovery#228
srnand wants to merge 2 commits into
mainfrom
srnand/may20_26/UseVeniceStoreMetadataFetcher

Conversation

@srnand
Copy link
Copy Markdown
Collaborator

@srnand srnand commented May 21, 2026

Summary

  • Switched Venice JDBC store discovery from ControllerClient based, cluster-by-cluster listing to RouterBasedStoreMetadataFetcher (single router /stores call).
  • This PR takes the approach over the OSS HTTP/HTTPS route, which is the only transport available in non-D2 deployments.
  • Bumps venice-common and venice-thin-client to 0.4.858, which is the first version exposing RouterBasedStoreMetadataFetcher.

Tests

  • Added UTs.

@srnand srnand requested a review from jogrogan May 21, 2026 01:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Code Coverage

Overall Project 84.4% -0.08% 🟢
Files changed 77.42% 🟢

File Coverage
ClusterSchema.java 86.22% -13.78% 🟢

Copy link
Copy Markdown
Collaborator

@ryannedolan ryannedolan left a comment

Choose a reason for hiding this comment

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

Nice!

Sign up for free to 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.

3 participants