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

fix(ingest/qlik): improve logging for debug (#10659)

Browse Source
This commit is contained in:
Aseem Bansal 2024-06-07 17:50:49 +05:30 committed by GitHub
parent ca88691ff4
commit 6fdf2f7354
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -93,6 +93,8 @@ class Constant:
# Personal entity constants
PERSONAL_SPACE_ID = "personal-space-id"
PERSONAL_SPACE_NAME = "personal_space"
# Hypercube
HYPERCUBE = "qHyperCube"
@dataclass

4
metadata-ingestion/src/datahub/ingestion/source/qlik_sense/data_classes.py
View File

@ -164,8 +164,8 @@ class Chart(BaseModel):
@root_validator(pre=True)
def update_values(cls, values: Dict) -> Dict:
values[Constant.QID] = values[Constant.QINFO][Constant.QID]
values["qDimension"] = values["qHyperCube"]["qDimensionInfo"]
values["qMeasure"] = values["qHyperCube"]["qMeasureInfo"]
values["qDimension"] = values[Constant.HYPERCUBE]["qDimensionInfo"]
values["qMeasure"] = values[Constant.HYPERCUBE]["qMeasureInfo"]
return values

13
metadata-ingestion/src/datahub/ingestion/source/qlik_sense/qlik_api.py
View File

@ -113,7 +113,13 @@ class QlikAPI:
method="GetChild", params={"qId": chart_id}
)
response = websocket_connection.websocket_send_request(method="GetLayout")
return Chart.parse_obj(response[Constant.QLAYOUT])
q_layout = response[Constant.QLAYOUT]
if Constant.HYPERCUBE not in q_layout:
logger.warning(
f"Chart with id {chart_id} of sheet {sheet_id} does not have hypercube. q_layout: {q_layout}"
)
return None
return Chart.parse_obj(q_layout)
except Exception as e:
self._log_http_error(
message=f"Unable to fetch chart {chart_id} of sheet {sheet_id}. Exception: {e}"
@ -135,6 +141,11 @@ class QlikAPI:
# That means sheet is private sheet
return None
sheet = Sheet.parse_obj(sheet_dict[Constant.QMETA])
if Constant.QCHILDLIST not in sheet_dict:
logger.warning(
f"Sheet {sheet.title} with id {sheet_id} does not have any charts. sheet_dict: {sheet_dict}"
)
return sheet
for i, chart_dict in enumerate(
sheet_dict[Constant.QCHILDLIST][Constant.QITEMS]
):