fix: remove redundant DROP INDEX in MySQL _drop_table_and_index

YassinNouh21 · YassinNouh21 · commit e7820903daf2 · 2026-04-07T11:13:35.000+02:00
MySQL DROP TABLE already removes all indexes. The separate DROP INDEX
was failing on versioned tables that were created via online_write_batch
(which doesn't create the _ek index). Also reuse _drop_table_and_index
in _drop_all_version_tables instead of duplicating logic.

Signed-off-by: yassinnouh21 &lt;yassinnouh21@gmail.com&gt;
diff --git a/sdk/python/feast/infra/online_stores/mysql_online_store/mysql.py b/sdk/python/feast/infra/online_stores/mysql_online_store/mysql.py
@@ -310,7 +310,6 @@ def teardown(
 
 
 def _drop_table_and_index(cur: Cursor, table_name: str) -> None:
-    cur.execute(f"DROP INDEX {table_name}_ek ON {table_name};")
     cur.execute(f"DROP TABLE IF EXISTS {table_name}")
 
 
@@ -323,8 +322,7 @@ def _drop_all_version_tables(cur: Cursor, project: str, table: FeatureView) -> N
         (base, f"^{base}_v[0-9]+$"),
     )
     for (name,) in cur.fetchall():
-        cur.execute(f"DROP INDEX IF EXISTS {name}_ek ON {name};")
-        cur.execute(f"DROP TABLE IF EXISTS {name}")
+        _drop_table_and_index(cur, name)
 
 
 def _table_id(project: str, table: FeatureView, enable_versioning: bool = False) -> str:
