-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Labels
Description
Expected Behavior
apply_materialization for Remote Registry should work for Stream Feature Views. Discovered as part of #4257 when running integration tests.
Current Behavior
apply_materialization for Remote Registry throws the below error when trying to materialize a stream feature view:
TypeError: Parameter to MergeFrom() must be instance of same class: expected <class 'feast.core.FeatureView_pb2.FeatureView'> got <class 'feast.core.StreamFeatureView_pb2.StreamFeatureView'>.
Steps to reproduce
- Setup Remote Registry
- Call
apply_materializationand pass a stream feature view as the feature view.
Specifications
- Version: Python 3.9
- Platform: Mac OSX
- Subsystem: Sonoma 14.4.1
Possible Solution
ApplyMaterializationRequest of the RegistryServer proto should be updated to allow the StreamFeatureView proto to be passed when making the request from Remote Registry via RegistryServer.