Merge pull request #445 from dbt-labs/fix-allow-description-larger-st…

…ring Allow larger strings for descriptions
dbt-labs · Apr 11, 2024 · cf65455 · cf65455
2 parents f390535 + e919211
commit cf65455
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/macros/unpack/type_large_string.sql b/macros/unpack/type_large_string.sql
@@ -0,0 +1,11 @@
+{%- macro type_large_string() -%}
+    {{ return(adapter.dispatch('type_large_string', 'dbt_project_evaluator')()) }}
+{%- endmacro -%}
+
+{%- macro default__type_large_string() -%}
+    {{ return(dbt.type_string()) }}
+{%- endmacro -%}
+
+{%- macro redshift__type_large_string() -%}
+    varchar(5000)
+{%- endmacro -%}
diff --git a/models/staging/graph/base/base_node_columns.sql b/models/staging/graph/base/base_node_columns.sql
@@ -16,7 +16,7 @@ with dummy_cte as (
 select 
     cast(null as {{ dbt.type_string() }}) as node_unique_id,
     cast(null as {{ dbt.type_string()}}) as name,
-    cast(null as {{ dbt.type_string()}}) as description,
+    cast(null as {{ dbt_project_evaluator.type_large_string()}}) as description,
     cast(null as {{ dbt.type_string()}}) as data_type,
     cast(null as {{ dbt.type_string()}}) as quote
 

diff --git a/models/staging/graph/base/base_source_columns.sql b/models/staging/graph/base/base_source_columns.sql
@@ -16,7 +16,7 @@ with dummy_cte as (
 select 
     cast(null as {{ dbt.type_string() }}) as node_unique_id,
     cast(null as {{ dbt.type_string()}}) as name,
-    cast(null as {{ dbt.type_string()}}) as description,
+    cast(null as {{ dbt_project_evaluator.type_large_string()}}) as description,
     cast(null as {{ dbt.type_string()}}) as data_type,
     cast(null as {{ dbt.type_string()}}) as quote