Skip to content

feat: Feature View Field powered to define Vector length instead of Config #5289

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.

Sign up for GitHub

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

Merged
franciscojavierarceo merged 10 commits into from
Apr 23, 2025
Merged

feat: Feature View Field powered to define Vector length instead of Config #5289

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
6038aac
No vector length - Postgress
jyejare Apr 16, 2025
93cb9f5
Field of Vector length
jyejare Apr 16, 2025
1c65906
SQLite vector length
jyejare Apr 21, 2025
bdcc759
ElasticSearch vector length
jyejare Apr 21, 2025
f4118dc
Qdrant Vector length
jyejare Apr 21, 2025
df15f5a
Test vector_length updates and related Fixes
jyejare Apr 21, 2025
1e7ef9a
Vector length cleanup for Store confgis
jyejare Apr 22, 2025
48e4184
All Protos regenerated
jyejare Apr 22, 2025
b4742f6
Vector len param renamed to Vector length
jyejare Apr 23, 2025
b431074
On Demand feature view transformation updated
jyejare Apr 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Vector length cleanup for Store confgis 
Signed-off-by: jyejare <jyejare@redhat.com>
  • Loading branch information
@jyejare
jyejare committed Apr 23, 2025
commit 1e7ef9ad65e12e507e22c59127edac53422a253f
1 change: 0 additions & 1 deletion sdk/python/feast/infra/online_stores/sqlite.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,6 @@ class SqliteOnlineStoreConfig(FeastConfigBaseModel, VectorStoreConfig):
""" (optional) Path to sqlite db """

vector_enabled: bool = False
vector_len: Optional[int] = None
text_search_enabled: bool = False


Expand Down
5 changes: 2 additions & 3 deletions sdk/python/tests/unit/test_on_demand_python_transformation.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1127,8 +1127,7 @@ def test_stored_writes_with_explode(self):
entity_key_serialization_version=3,
online_store=SqliteOnlineStoreConfig(
path=os.path.join(data_dir, "online.db"),
vector_enabled=True,
vector_len=5,
vector_enabled=True
),
)
)
Expand Down Expand Up @@ -1378,7 +1377,6 @@ def test_docling_transform(self):
online_store=SqliteOnlineStoreConfig(
path=os.path.join(data_dir, "online.db"),
vector_enabled=True,
vector_len=VECTOR_LEN,
),
)
)
Expand Down Expand Up @@ -1422,6 +1420,7 @@ def embed_chunk(input_string) -> dict[str, list[float]]:
name="vector",
dtype=Array(Float32),
vector_index=True,
vector_len=VECTOR_LEN,
vector_search_metric="L2",
),
],
Expand Down