Providing warning instead of raising error due to chances of breaking other functionalities

aniketpalu · ntkathole · commit d6f4b4ed114f · 2025-07-23T19:15:24.000+05:30
Signed-off-by: Aniket Paluskar &lt;apaluska@redhat.com&gt;
diff --git a/sdk/python/feast/feature_store.py b/sdk/python/feast/feature_store.py
@@ -1900,14 +1900,14 @@ def write_to_online_store(
         # Validate that the dataframe has meaningful feature data
         if df is not None:
             if df.empty:
-                raise ValueError("Cannot write empty dataframe to online store")
+                warnings.warn("Cannot write empty dataframe to online store")
 
             # Check if feature columns are empty (entity columns may have data but feature columns are empty)
             feature_column_names = [f.name for f in feature_view.features]
             if feature_column_names:
                 feature_df = df[feature_column_names]
                 if feature_df.empty or feature_df.isnull().all().all():
-                    raise ValueError(
+                    warnings.warn(
                         "Cannot write dataframe with empty feature columns to online store"
                     )
 
@@ -1944,14 +1944,14 @@ async def write_to_online_store_async(
         # Validate that the dataframe has meaningful feature data
         if df is not None:
             if df.empty:
-                raise ValueError("Cannot write empty dataframe to online store")
+                warnings.warn("Cannot write empty dataframe to online store")
 
             # Check if feature columns are empty (entity columns may have data but feature columns are empty)
             feature_column_names = [f.name for f in feature_view.features]
             if feature_column_names:
                 feature_df = df[feature_column_names]
                 if feature_df.empty or feature_df.isnull().all().all():
-                    raise ValueError(
+                    warnings.warn(
                         "Cannot write dataframe with empty feature columns to online store"
                     )
 
