ClickHouse / dbt-clickhouse

The Clickhouse plugin for dbt (data build tool)
Apache License 2.0
254 stars 113 forks source link

Update persist_docs.sql #313

Closed GavrG closed 4 months ago

GavrG commented 4 months ago

Fixes Issue Table and Column Comments set up misbehaviour #295

CLAassistant commented 4 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

BentsiLeviav commented 4 months ago

Hi @GavrG

Thank you for your contribution. Why splitting the commends is supposed to solve the issue? (I tested it and the problem still occurs) The suggested solution doesn't resolve the problem as it is a ClickHouse issue and not a dbt (adapter) issue (the responsibility of syncing nodes' metadata).

The workaround I would suggest is to use clusterAllReplicas in the macro, which queries the entire cluster.