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

fix(ingest): fix minor bug + protective dep requirements (#7861)

Browse Source
This commit is contained in:
Harshal Sheth 2023-04-26 03:05:01 +05:30 committed by GitHub
parent 031aee4298
commit 29e5cfd643
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
metadata-ingestion/setup.py
View File

@ -25,7 +25,8 @@ base_requirements = {
# Actual dependencies.
"typing-inspect",
# pydantic 1.10.3 is incompatible with typing-extensions 4.1.1 - https://github.com/pydantic/pydantic/issues/4885
"pydantic>=1.5.1,!=1.10.3",
# pydantic 2 makes major, backwards-incompatible changes - https://github.com/pydantic/pydantic/issues/4887
"pydantic>=1.5.1,!=1.10.3,<2",
"mixpanel>=4.9.0",
}

1
metadata-ingestion/src/datahub/cli/ingest_cli.py
View File

@ -199,6 +199,7 @@ def run(
def _test_source_connection(report_to: Optional[str], pipeline_config: dict) -> None:
connection_report = None
try:
connection_report = ConnectionManager().test_source_connection(pipeline_config)
logger.info(connection_report.as_json())

3
metadata-ingestion/src/datahub/ingestion/source/looker/looker_lib_wrapper.py
View File

@ -14,6 +14,7 @@ from looker_sdk.sdk.api40.models import (
DBConnection,
Folder,
LookmlModel,
LookmlModelExplore,
User,
WriteQuery,
)
@ -145,7 +146,7 @@ class LookerAPI:
transport_options=self.transport_options,
)
def lookml_model_explore(self, model, explore_name):
def lookml_model_explore(self, model: str, explore_name: str) -> LookmlModelExplore:
self.client_stats.explore_calls += 1
return self.client.lookml_model_explore(
model, explore_name, transport_options=self.transport_options