Skip to content

Commit 68486e5

Browse files
authored
Merge pull request #475 from mweso-softserve/type_string_macro_handling_all_cases
2 parents 1e66191 + a4f2a82 commit 68486e5

15 files changed

Lines changed: 92 additions & 84 deletions

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ Currently, the following adapters are supported:
2929

3030
Check [dbt Hub](https://hub.getdbt.com/dbt-labs/dbt_project_evaluator/latest/) for the latest installation instructions, or [read the docs](https://docs.getdbt.com/docs/package-management) for more information on installing packages.
3131

32-
### Additional setup for Databricks/Spark/DuckDB/Redshift
32+
### Additional setup for Databricks/Spark/DuckDB
3333

3434
In your `dbt_project.yml`, add the following config:
3535

macros/cross_db_shim/redshift_shims.sql

Lines changed: 0 additions & 3 deletions
This file was deleted.
File renamed without changes.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{%- macro type_string_dpe() -%}
2+
{{ return(adapter.dispatch('type_string_dpe', 'dbt_project_evaluator')()) }}
3+
{%- endmacro -%}
4+
5+
{%- macro default__type_string_dpe() -%}
6+
{{ return(dbt.type_string()) }}
7+
{%- endmacro -%}
8+
9+
{%- macro redshift__type_string_dpe() -%}
10+
{{ return(api.Column.string_type(600)) }}
11+
{%- endmacro -%}

macros/unpack/get_relationship_values.sql

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@
2323

2424
{%- set values_line =
2525
[
26-
"cast('" ~ node.unique_id ~ "' as " ~ dbt.type_string() ~ ")",
27-
"cast(NULL as " ~ dbt.type_string() ~ ")",
26+
"cast('" ~ node.unique_id ~ "' as " ~ dbt_project_evaluator.type_string_dpe() ~ ")",
27+
"cast(NULL as " ~ dbt_project_evaluator.type_string_dpe() ~ ")",
2828
"FALSE",
2929
]
3030
%}
@@ -37,8 +37,8 @@
3737

3838
{%- set values_line =
3939
[
40-
"cast('" ~ node.unique_id ~ "' as " ~ dbt.type_string() ~ ")",
41-
"cast('" ~ parent ~ "' as " ~ dbt.type_string() ~ ")",
40+
"cast('" ~ node.unique_id ~ "' as " ~ dbt_project_evaluator.type_string_dpe() ~ ")",
41+
"cast('" ~ parent ~ "' as " ~ dbt_project_evaluator.type_string_dpe() ~ ")",
4242
"" ~ loop.last ~ "" if node.unique_id.split('.')[0] == 'test' else "FALSE"
4343
]
4444
%}

macros/wrap_string_with_quotes.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{% macro wrap_string_with_quotes(str) %}
22
{% if str is none %}
3-
{{ return('cast(NULL as ' ~ dbt.type_string() ~ ')') }}
3+
{{ return('cast(NULL as ' ~ dbt_project_evaluator.type_string_dpe() ~ ')') }}
44
{% else %}
55
{{ dbt.string_literal(str) }}
66
{% endif %}

models/staging/graph/base/base_exposure_relationships.sql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ with dummy_cte as (
1515
)
1616

1717
select
18-
cast(null as {{ dbt.type_string()}}) as resource_id,
19-
cast(null as {{ dbt.type_string()}}) as direct_parent_id,
18+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as resource_id,
19+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as direct_parent_id,
2020
cast(True as boolean) as is_primary_relationship
2121

2222
from dummy_cte

models/staging/graph/base/base_metric_relationships.sql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ with dummy_cte as (
1515
)
1616

1717
select
18-
cast(null as {{ dbt.type_string()}}) as resource_id,
19-
cast(null as {{ dbt.type_string()}}) as direct_parent_id,
18+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as resource_id,
19+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as direct_parent_id,
2020
cast(True as boolean) as is_primary_relationship
2121

2222
from dummy_cte

models/staging/graph/base/base_node_columns.sql

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,14 @@ with dummy_cte as (
1414
)
1515

1616
select
17-
cast(null as {{ dbt.type_string() }}) as node_unique_id,
18-
cast(null as {{ dbt.type_string()}}) as name,
17+
cast(null as {{ dbt_project_evaluator.type_string_dpe() }}) as node_unique_id,
18+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as name,
1919
cast(null as {{ dbt_project_evaluator.type_large_string()}}) as description,
20-
cast(null as {{ dbt.type_string()}}) as data_type,
21-
cast(null as {{ dbt.type_string()}}) as constraints,
20+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as data_type,
21+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as constraints,
2222
cast(True as boolean) as has_not_null_constraint,
2323
cast(0 as {{ dbt.type_int() }}) as constraints_count,
24-
cast(null as {{ dbt.type_string()}}) as quote
24+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as quote
2525

2626
from dummy_cte
2727
where false

models/staging/graph/base/base_node_relationships.sql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ with dummy_cte as (
1414
)
1515

1616
select
17-
cast(null as {{ dbt.type_string()}}) as resource_id,
18-
cast(null as {{ dbt.type_string()}}) as direct_parent_id,
17+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as resource_id,
18+
cast(null as {{ dbt_project_evaluator.type_string_dpe()}}) as direct_parent_id,
1919
cast(True as boolean) as is_primary_relationship
2020

2121
from dummy_cte

0 commit comments

Comments
 (0)