feat(ingestion/tableau): optionally ingest multiple sites and create …

…site containers (datahub-project#10498) Co-authored-by: Yanik Häni <[email protected]>
aviv-julienjehannet · Jul 25, 2024 · 5bcf85b · 5bcf85b
1 parent dc4cc64
commit 5bcf85b
Show file tree

Hide file tree

Showing 9 changed files with 175,591 additions and 121 deletions.
diff --git a/docs/how/updating-datahub.md b/docs/how/updating-datahub.md
@@ -32,6 +32,7 @@ This file documents any backwards-incompatible changes in DataHub and assists pe
   No loss of functionality expected unless explicitly mentioned in Breaking Changes.
 
 ### Other Notable Changes
+- #10498 - Tableau ingestion can now be configured to ingest multiple sites at once and add the sites as containers. The feature is currently only available for Tableau Server.
 
 ## 0.13.3
 

diff --git a/metadata-ingestion/src/datahub/ingestion/source/tableau.py b/metadata-ingestion/src/datahub/ingestion/source/tableau.py
diff --git a/metadata-ingestion/src/datahub/ingestion/source/tableau_common.py b/metadata-ingestion/src/datahub/ingestion/source/tableau_common.py
@@ -767,7 +767,12 @@ def get_overridden_info(
         ):
             platform_instance = database_hostname_to_platform_instance_map.get(hostname)
 
-    if original_platform in ("athena", "hive", "mysql"):  # Two tier databases
+    if original_platform in (
+        "athena",
+        "hive",
+        "mysql",
+        "teradata",
+    ):  # Two tier databases
         upstream_db = None
 
     return upstream_db, platform_instance, platform, original_platform

diff --git a/metadata-ingestion/src/datahub/ingestion/source/tableau_constant.py b/metadata-ingestion/src/datahub/ingestion/source/tableau_constant.py
@@ -78,4 +78,5 @@
 DASHBOARDS_CONNECTION = "dashboardsConnection"
 EMBEDDED_DATA_SOURCES_CONNECTION = "embeddedDatasourcesConnection"
 PROJECT = "Project"
+SITE = "Site"
 IS_UNSUPPORTED_CUSTOM_SQL = "isUnsupportedCustomSql"