Skip to content

Commit

Permalink
Merge pull request #257 from ClickHouse/fix-settings-custom-mat
Browse files Browse the repository at this point in the history
Fix model settings to use a custom materialization
  • Loading branch information
BentsiLeviav committed Mar 26, 2024
2 parents 62efd62 + 23b01e8 commit 0c44464
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion dbt/adapters/clickhouse/dbclient.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,8 @@ def _server_version(self):
pass

def update_model_settings(self, model_settings: Dict[str, str], materialization_type: str):
model_settings_to_add = copy.deepcopy(self._model_settings[materialization_type])
settings = self._model_settings.get(materialization_type, {})
model_settings_to_add = copy.deepcopy(settings)
model_settings_to_add.update(self._model_settings['general'])
for key, value in model_settings_to_add.items():
if key not in model_settings:
Expand Down

0 comments on commit 0c44464

Please sign in to comment.