← 输入框|github.com|刷新
Skip to content

feat: Enable routing strategies in glide redis client - #7079

Open
jakes44 wants to merge 2 commits into
feast-dev:masterfrom
jakes44:master
Open

jakes44 wants to merge 2 commits into
feast-dev:masterfrom
jakes44:master

Conversation

@jakes44

@jakes44 jakes44 commented Oct 27, 2026

Copy link
Copy Markdown

Parse routing strategy and availability zone from the feature_store.taml connection string to pass to the glide client configuration

What this PR does / why we need it:

For Valkey instances with replicas, it's important to have routing options to minimize network hops and relieve load from the primary/spread load on read heavvy operations. This is already supported in the glide client, and now adding as an option to feast.

Checks

  • [ x ] I've made sure the tests are passing.
  • [ x ] My commits are signed off (git commit -s)
  • [ x ] My PR title follows conventional commits format

Testing Strategy

  • [ x ] Unit tests
  • Integration tests
  • Manual tests
  • Testing is not required for this change

Parse routing strategy and availability zone from the feature_store.taml connection string to pass to the glide client configuration

Signed-off-by: Jesse Akes <jakes44@users.noreply.github.com>
@jakes44
jakes44 requested a review from a team as a code owner October 27, 2026 18:45
@jakes44

jakes44 commented Oct 27, 2026

Copy link
Copy Markdown
Author

cc: @chandlerok

@jakes44

jakes44 commented Oct 28, 2026

Copy link
Copy Markdown
Author

And @ntkathole for review

Accidental lint add

Signed-off-by: Jesse Akes <akesjesse@gmail.com>

This branch has not been deployed

No deployments
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.

1 participant