fix(spark): narrow exception handling in offline path fallback

abhijeet-dhumal · ntkathole · commit 17405f326f0f · 2026-07-20T14:15:49.000+05:30
Catch FileNotFoundError and PermissionError separately for the expected
fallback cases (path not yet materialized, or no access). Unexpected
errors now emit a distinct RuntimeWarning instead of being silently
swallowed by a bare except Exception.

Signed-off-by: abhijeet-dhumal &lt;abhijeetdhumal652@gmail.com&gt;
diff --git a/sdk/python/feast/infra/offline_stores/contrib/spark_offline_store/spark.py b/sdk/python/feast/infra/offline_stores/contrib/spark_offline_store/spark.py
@@ -131,10 +131,17 @@ def _apply_bfv_transformations_for_historical(
                 ctx = replace(ctx, table_subquery=tmp_view)
                 new_contexts.append(ctx)
                 continue
-            except Exception:
+            except (FileNotFoundError, PermissionError) as e:
                 warnings.warn(
-                    f"Offline path '{fv.batch_source.path}' not readable for "
-                    f"'{ctx.name}'; falling back to source query.",
+                    f"Offline path '{fv.batch_source.path}' not accessible for "
+                    f"'{ctx.name}': {e}; falling back to source query.",
+                    RuntimeWarning,
+                    stacklevel=2,
+                )
+            except Exception as e:
+                warnings.warn(
+                    f"Unexpected error loading offline path '{fv.batch_source.path}' "
+                    f"for '{ctx.name}': {e}; falling back to source query.",
                     RuntimeWarning,
                     stacklevel=2,
                 )
