Skip to content

Commit

Permalink
Update teradata.py
Browse files Browse the repository at this point in the history
  • Loading branch information
brock-acryl committed Feb 7, 2025
1 parent 40499e0 commit db2deed
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -684,7 +684,7 @@ def _add_default_options(self, sql_config: SQLCommonConfig) -> None:
super()._add_default_options(sql_config)
if sql_config.is_profiling_enabled():

Check warning on line 685 in metadata-ingestion/src/datahub/ingestion/source/sql/teradata.py

View check run for this annotation

Codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/sql/teradata.py#L684-L685

Added lines #L684 - L685 were not covered by tests
# Sqlalchemy uses QueuePool by default however Teradata uses SingletonThreadPool.
# SingletonThreadPool does not support parellel connections.
# SingletonThreadPool does not support parellel connections. For using profiling, we need to use QueuePool.
# https://docs.sqlalchemy.org/en/20/core/pooling.html#connection-pool-configuration
# https://github.com/Teradata/sqlalchemy-teradata/issues/96
sql_config.options.setdefault("poolclass", QueuePool)

Check warning on line 690 in metadata-ingestion/src/datahub/ingestion/source/sql/teradata.py

View check run for this annotation

Codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/sql/teradata.py#L690

Added line #L690 was not covered by tests
Expand Down

0 comments on commit db2deed

Please sign in to comment.