Addressed Integration Tests Issues

databrickslabs · Nov 4, 2024 · 238cefd · 238cefd
1 parent acec522
commit 238cefd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/databricks/labs/ucx/contexts/application.py b/src/databricks/labs/ucx/contexts/application.py
@@ -284,7 +284,7 @@ def static_table_ownership(self) -> StaticTableOwnership:
             self.administrator_locator,
             self.tables_crawler,
             self.config.default_owner_group,
-            self.connect_config.username
+            self.connect_config.username,
         )
 
     @cached_property

diff --git a/src/databricks/labs/ucx/hive_metastore/ownership.py b/src/databricks/labs/ucx/hive_metastore/ownership.py
@@ -109,6 +109,9 @@ def load(self) -> Iterable[Grant]:
         for table in self._tables_crawler.snapshot():
             owner = self._maybe_direct_owner(table)
             table_name, view_name = self._names(table)
+            if not owner:
+                logger.warning(f"No owner found for {table.key}")
+                continue
             yield Grant(
                 principal=owner,
                 action_type='OWN',