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

fix(ingest/looker): add clarity in chart input parsing logs (#7003)

Browse Source
This commit is contained in:
Harshal Sheth 2023-01-12 20:40:19 -08:00 committed by GitHub
parent 8e920717aa
commit 9579e69170
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
metadata-ingestion/src/datahub/ingestion/source/looker/looker_source.py
View File

@ -474,7 +474,7 @@ class LookerDashboardSource(TestableSource, StatefulIngestionSourceBase):
# Get the explore from the view directly
explores = [element.query.view] if element.query.view is not None else []
logger.debug(
"Element {}: Explores added: {}".format(element.title, explores)
f"Element {element.title}: Explores added via query: {explores}"
)
for exp in explores:
self.add_explore_to_fetch(
@ -511,7 +511,7 @@ class LookerDashboardSource(TestableSource, StatefulIngestionSourceBase):
input_fields = self._get_input_fields_from_query(element.look.query)
if element.look.query.view is not None:
explores = [element.look.query.view]
logger.debug(f"Element {title}: Explores added: {explores}")
logger.debug(f"Element {title}: Explores added via look: {explores}")
for exp in explores:
self.add_explore_to_fetch(
model=element.look.query.model,
@ -552,7 +552,7 @@ class LookerDashboardSource(TestableSource, StatefulIngestionSourceBase):
)
logger.debug(
"Element {}: Explores added: {}".format(element.title, explores)
f"Element {element.title}: Explores added via result_maker: {explores}"
)
for exp in explores:
@ -605,6 +605,7 @@ class LookerDashboardSource(TestableSource, StatefulIngestionSourceBase):
input_fields=input_fields,
)
logger.debug(f"Element {element.title}: Unable to parse LookerDashboardElement")
return None
def _get_chart_type(