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

fix: Operator - make onlineStore the default service #5044

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tchughesiv
Copy link
Contributor

@tchughesiv tchughesiv commented Feb 11, 2025

What this PR does / why we need it:

Currently, the default deployed feast server container is the remote registry grpc server. With this change, the default will change to an online feature server container. The user can still run an offline and/or registry server container if they like.

The reasoning behind this change is as follows:

  • The online feature server is the more commonly used / necessary component of a FeatureStore.
  • The online feature server is a read-only service, so is more secure out of the box than a remote registry service. As such, running the grpc registry should be an explicit choice by the user.
  • The remote registry provider type and server is a still relatively new feature and more of an edge use-case

@tchughesiv tchughesiv force-pushed the onlineDefault branch 5 times, most recently from 4a35aa5 to a0f9d75 Compare February 12, 2025 17:14
@tchughesiv tchughesiv marked this pull request as ready for review February 12, 2025 21:39
@tchughesiv tchughesiv requested a review from a team as a code owner February 12, 2025 21:39
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.

1 participant