feat: Adding features field to retrieve_online_features to return mor… #4869
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
This pull request introduces several enhancements and refactorings to the
retrieve_online_documents
method and related components within the Feast SDK. The primary changes include the addition of support for multiple features to be retrieved from the online document store, along with improvements to the serialization and deserialization of float vectors.Changes
Feature Store Enhancements
retrieve_online_documents
to accept an optional list of features (features
).Utility Functions
_serialize_vector_to_float_list
function infeast/utils.py
for serializing float vectors toValueProto
.Key Encoding Utilities
serialize_f32
anddeserialize_f32
functions infeast/infra/key_encoding_utils.py
for efficient serialization and deserialization of float lists.Provider and Online Store Changes
passthrough_provider.py
,faiss_online_store.py
,sqlite.py
,postgres.py
,elasticsearch.py
,qdrant.py
) to handle the newrequested_features
parameter.requested_feature
parameter optional.Refactoring
serve_registry
method definition for better readability.Testing
Documentation
Which issue(s) this PR fixes:
Step along the path to ship Milvus (#4364)
Misc
Related to this: #4751