Skip to content

Commit 79cb8e1

Browse files
removing commenting and unnecessary code
Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
1 parent afc2bb6 commit 79cb8e1

File tree

2 files changed

+4
-13
lines changed

2 files changed

+4
-13
lines changed

sdk/python/feast/infra/online_stores/sqlite.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -176,14 +176,6 @@ def online_write_batch(
176176
) # type: ignore
177177
else:
178178
val_bin = feast_value_type_to_python_type(val)
179-
# print(
180-
# f"entity_key_bin = {entity_key_bin} (type: {type(entity_key_bin)})\n"
181-
# f"feature_name = {feature_name} (type: {type(feature_name)})\n"
182-
# f"value = {val.SerializeToString()} (type: {type(val.SerializeToString())})\n"
183-
# f"val_bin = {val_bin} (type: {type(val_bin)})\n"
184-
# f"timestamp = {timestamp} (type: {type(timestamp)})\n"
185-
# f"created_ts = {created_ts} (type: {type(created_ts)})\n"
186-
# )
187179
conn.execute(
188180
f"""
189181
INSERT INTO {table_name} (entity_key, feature_name, value, vector_value, event_ts, created_ts)
@@ -489,7 +481,6 @@ def retrieve_online_documents_v2(
489481
raise ValueError("vector_len is not configured in the online store config")
490482

491483
table_name = _table_id(config.project, table)
492-
# Currently this is the problem
493484
vector_field = _get_vector_field(table)
494485

495486
if online_store.vector_enabled:

sdk/python/tests/unit/online_store/test_online_retrieval.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -855,10 +855,10 @@ def test_sqlite_hybrid_search() -> None:
855855
res_df["title"].tolist() == ["Guardians of the Galaxy", "Sing", "Suicide Squad"]
856856

857857

858-
# @pytest.mark.skipif(
859-
# sys.version_info[0:2] != (3, 10),
860-
# reason="Only works on Python 3.10",
861-
# )
858+
@pytest.mark.skipif(
859+
sys.version_info[0:2] != (3, 10),
860+
reason="Only works on Python 3.10",
861+
)
862862
def test_sqlite_get_online_documents_v2() -> None:
863863
"""Test retrieving documents using v2 method with vector similarity search."""
864864
n = 10

0 commit comments

Comments
 (0)