Merge pull request #28 from ExpediaGroup/debug_apply

fixc linting
ExpediaGroup · Aug 31, 2023 · 33b16ce · 33b16ce
2 parents d2e2071 + b7409d7
commit 33b16ce
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/sdk/python/feast/feature_store.py b/sdk/python/feast/feature_store.py
@@ -821,7 +821,8 @@ def apply(
             ... )
             >>> fs.apply([driver_hourly_stats_view, driver]) # register entity and feature view
         """
-        print(f"Apply with: {objects}") #TODO: remove
+        # TODO: remove
+        print(f"Apply with: {objects}")
         # TODO: Add locking
         if not isinstance(objects, Iterable):
             objects = [objects]
@@ -992,12 +993,15 @@ def apply(
         tables_to_delete: List[FeatureView] = views_to_delete + sfvs_to_delete if not partial else []  # type: ignore
         tables_to_keep: List[FeatureView] = views_to_update + sfvs_to_update  # type: ignore
 
-        print(f"""Update project={self.project},
+        # TODO: remove
+        print(
+            f"""Update project={self.project},
             tables_to_delete={tables_to_delete},
             tables_to_keep={tables_to_keep},
             entities_to_delete={entities_to_delete if not partial else []},
             entities_to_keep={entities_to_update},
-            partial={partial},""") #TODO: remove
+            partial={partial},"""
+        )
         self._get_provider().update_infra(
             project=self.project,
             tables_to_delete=tables_to_delete,

diff --git a/sdk/python/feast/infra/passthrough_provider.py b/sdk/python/feast/infra/passthrough_provider.py
@@ -116,7 +116,8 @@ def update_infra(
     ):
         set_usage_attribute("provider", self.__class__.__name__)
 
-        print(f"Is online store={self.online_store}") #TODO: remove
+        # TODO: remove
+        print(f"Is online store={self.online_store}")
         # Call update only if there is an online store
         if self.online_store:
             self.online_store.update(