Skip to content

feat: Extend enable/disable/set-state to all feature view types and REST/gRPC APIs#6580

Open
Rakeshpatil01 wants to merge 2 commits into
feast-dev:masterfrom
Rakeshpatil01:feature/issue-6429
Open

feat: Extend enable/disable/set-state to all feature view types and REST/gRPC APIs#6580
Rakeshpatil01 wants to merge 2 commits into
feast-dev:masterfrom
Rakeshpatil01:feature/issue-6429

Conversation

@Rakeshpatil01

@Rakeshpatil01 Rakeshpatil01 commented Jul 2, 2026

Copy link
Copy Markdown

1. What this PR does / why we need it:

This PR resolves issue #6429 by extending the enable_feature_view,
disable_feature_view, and set_feature_view_state operations to support all
feature view types, specifically adding support for StreamFeatureView. It also
exposes these actions through new REST and gRPC API endpoints for external
clients.

2. Changes:

  • gRPC Server: Added EnableFeatureView, DisableFeatureView, and
    SetFeatureViewState handlers in RegistryServer to enable operations from
    gRPC clients.

  • REST API Router: Added FastAPI routes (/feature_views/{name}/enable,
    /disable, /state) delegating requests to the gRPC client.

  • Core FeatureStore: Enhanced client validation in feature_store.py to
    support StreamFeatureView type validation and raise clean ValueError
    exceptions for unsupported resource types (like Entities).

  • CLI Groups: Added enable, disable, and set-state subcommands to
    on-demand-feature-views and stream-feature-views Click command groups.

  • Protos: Compiled updated proto interfaces in RegistryServer.proto.

    Which issue(s) this PR fixes:

    Fixes Extend enable/disable/set-state to all feature view types and REST/gRPC APIs #6429

    Checks

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

    Testing Strategy

    • Unit tests
    • Integration tests

    Testing details:

    Executed the unit tests and REST API integration tests:

    uv run python -m pytest sdk/python/tests/unit/test_feature_view_state.py
    sdk/python/tests/unit/api/test_api_rest_registry.py
    
    Results: All  101 passed  successfully in 33.48 seconds.

…EST/gRPC APIs

Signed-off-by: Rakeshpatil01 <patilrp1996@gmail.com>
@Rakeshpatil01 Rakeshpatil01 requested a review from a team as a code owner July 2, 2026 13:50
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.

Extend enable/disable/set-state to all feature view types and REST/gRPC APIs

1 participant