Merge pull request #87 from ClickHouse/fix_native_settings

Fix ClickHouse settings issue with native driver
ClickHouse · Aug 1, 2022 · 0952cf5 · 0952cf5
2 parents 1d23e88 + 14b38df
commit 0952cf5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/dbt/adapters/clickhouse/connections.py b/dbt/adapters/clickhouse/connections.py
@@ -280,7 +280,7 @@ def _connect_native(credentials):
             sync_request_timeout=credentials.sync_request_timeout,
             compress_block_size=credentials.compress_block_size,
             compression=False if credentials.compression == '' else credentials.compression,
-            **(credentials.custom_settings or {}),
+            settings=credentials.custom_settings,
         )
         client = ChNativeAdapter(client)
         db_err = _ensure_database(client, credentials.schema)

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -58,6 +58,7 @@ def dbt_profile_target():
         'port': int(os.environ.get('PORT_ENV_VAR_NAME', 8123)),  # docker client port
         'secure': False,
         'driver': driver,
+        'custom_settings': {'distributed_ddl_task_timeout': 300},
     }