Skip to content

fix: Add remote registry client extra - #6697

Open
yaodong-shen wants to merge 1 commit into
feast-dev:masterfrom
yaodong-shen:fix/remote-registry-client-extra
Open

fix: Add remote registry client extra#6697
yaodong-shen wants to merge 1 commit into
feast-dev:masterfrom
yaodong-shen:fix/remote-registry-client-extra

Conversation

@yaodong-shen

Copy link
Copy Markdown

What this PR does

Fixes #6666. RemoteRegistry imports grpc unconditionally, but grpcio was only available through the unrelated server-oriented grpcio extra. Users installing a normal provider extra could therefore configure registry_type: remote and receive ModuleNotFoundError at runtime.

This adds a focused remote optional dependency containing only grpcio and documents pip install "feast[remote]" next to the remote registry configuration. The existing grpcio extra remains unchanged for registry/feature-server deployments that need reflection and health-checking packages.

Validation

  • Verified the final branch diff contains only pyproject.toml and docs/reference/registries/remote.md.
  • Verified the TOML extra and documentation text from the master blobs.
  • Full Feast dependency installation and unit test suite were not runnable locally because the network clone did not complete; CI should exercise package metadata and docs checks.

Release notes

NONE

@yaodong-shen
yaodong-shen requested a review from a team as a code owner August 1, 2026 17:05
@yaodong-shen

Copy link
Copy Markdown
Author

The implementation is ready and the branch is mergeable. Per the repository contribution guide, could a maintainer add the required kind/bug and ok-to-test labels so the Prow checks can run? The DCO check still correctly awaits the contributor's explicit sign-off authorization.

@yaodong-shen
yaodong-shen force-pushed the fix/remote-registry-client-extra branch from ca3823a to 9e9ee68 Compare August 2, 2026 01:03

@franciscojavierarceo franciscojavierarceo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The focused extra cleanly separates the client-only grpcio dependency and documents the installation path without changing the existing server extra. I found no blocking issue in the remote diff.

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.

Remote registry client fails with ModuleNotFoundError: No module named 'grpc' unless the unrelated [grpcio] extra is installed

2 participants