From ada40e6edd66931754f5e9761819742d48a38000 Mon Sep 17 00:00:00 2001 From: Akhil <62130268+akhil7philip@users.noreply.github.com> Date: Mon, 28 Aug 2023 15:03:43 +0530 Subject: [PATCH] fix(superset): get alternate platform value if sqlalchemy_uri param is missing (#8667) --- metadata-ingestion/src/datahub/ingestion/source/superset.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/metadata-ingestion/src/datahub/ingestion/source/superset.py b/metadata-ingestion/src/datahub/ingestion/source/superset.py index 7bf19db25e3bb..2a4563439b6ba 100644 --- a/metadata-ingestion/src/datahub/ingestion/source/superset.py +++ b/metadata-ingestion/src/datahub/ingestion/source/superset.py @@ -200,6 +200,8 @@ def get_platform_from_database_id(self, database_id): f"{self.config.connect_uri}/api/v1/database/{database_id}" ).json() sqlalchemy_uri = database_response.get("result", {}).get("sqlalchemy_uri") + if sqlalchemy_uri is None: + return database_response.get("result", {}).get("backend", "external") return sql_common.get_platform_from_sqlalchemy_uri(sqlalchemy_uri) @lru_cache(maxsize=None)