yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-12-02 05:33:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(ingest/tableau): load project workbook hierarchy correctly (#7483)

Browse Source
This commit is contained in:
Harshal Sheth 2023-03-03 14:54:42 -05:00 committed by GitHub
parent b4dd1d7d82
commit 795d76b56c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
metadata-ingestion/src/datahub/ingestion/source/tableau.py
View File

@ -2091,12 +2091,7 @@ class TableauSource(StatefulIngestionSourceBase):
]
)
dashboard_snapshot.aspects.append(browse_paths)
else:
logger.warning(
f"Could not set browse path for dashboard {dashboard[tableau_constant.ID]}. Please check permissions."
)
if (
elif (
workbook is not None
and workbook.get(tableau_constant.PROJECT_NAME)
and workbook.get(tableau_constant.NAME)
@ -2110,8 +2105,8 @@ class TableauSource(StatefulIngestionSourceBase):
)
dashboard_snapshot.aspects.append(browse_paths)
else:
logger.debug(
f"Browse path not set for dashboard {dashboard[tableau_constant.ID]}"
logger.warning(
f"Could not set browse path for dashboard {dashboard[tableau_constant.ID]}. Please check permissions."
)
# Ownership

2
metadata-ingestion/src/datahub/ingestion/source/tableau_common.py
View File

@ -85,6 +85,7 @@ sheet_graphql_query = """
id
name
projectName
projectLuid
owner {
username
}
@ -162,6 +163,7 @@ dashboard_graphql_query = """
id
name
projectName
projectLuid
owner {
username
}