Skip to content

Commit

Permalink
fix(ingest): mssql - lowercase db name in mssql ingestion (datahub-pr…
Browse files Browse the repository at this point in the history
  • Loading branch information
hsheth2 authored Nov 16, 2022
1 parent ba7fc3a commit cf6fb91
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions metadata-ingestion/src/datahub/ingestion/source/sql/mssql.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ def __init__(self, config: SQLServerConfig, ctx: PipelineContext):
super().__init__(config, ctx, "mssql")
# Cache the table and column descriptions
self.config: SQLServerConfig = config
self.current_database = None
self.current_database: Optional[str] = None
self.table_descriptions: Dict[str, str] = {}
self.column_descriptions: Dict[str, str] = {}
for inspector in self.get_inspectors():
Expand Down Expand Up @@ -262,5 +262,5 @@ def get_identifier(
return f"{self.config.database_alias}.{regular}"
return f"{self.config.database}.{regular}"
if self.current_database:
return f"{self.current_database}.{regular}"
return f"{self.current_database.lower()}.{regular}"
return regular

0 comments on commit cf6fb91

Please sign in to comment.