Skip to content

Commit

Permalink
Make type macro calls fully qualified (#232)
Browse files Browse the repository at this point in the history
* Make type macro calls fully qualified

* Update calls to dbt-core date macros
  • Loading branch information
clausherther committed Dec 28, 2022
1 parent 0a358e5 commit 8847b5a
Show file tree
Hide file tree
Showing 18 changed files with 43 additions and 43 deletions.
8 changes: 4 additions & 4 deletions integration_tests/models/schema_tests/data_test.sql
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
select
1 as idx,
'2020-10-21' as date_col,
cast(0 as {{ type_float() }}) as col_numeric_a,
cast(1 as {{ type_float() }}) as col_numeric_b,
cast(0 as {{ dbt.type_float() }}) as col_numeric_a,
cast(1 as {{ dbt.type_float() }}) as col_numeric_b,
'a' as col_string_a,
'b' as col_string_b,
cast(null as {{ type_string() }}) as col_null,
cast(null as {{ type_string() }}) as col_null_2
cast(null as {{ dbt.type_string() }}) as col_null,
cast(null as {{ dbt.type_string() }}) as col_null_2

union all

Expand Down
4 changes: 2 additions & 2 deletions integration_tests/models/schema_tests/timeseries_data.sql
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ add_row_values as (
select
cast(date_day as date) as date_day,
cast(date_day as {{ dbt_expectations.type_datetime() }}) as date_datetime,
cast(date_day as {{ type_timestamp() }}) as date_timestamp,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ type_float() }}) as row_value
cast(date_day as {{ dbt_expectations.type_timestamp() }}) as date_timestamp,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ dbt.type_float() }}) as row_value

from
dates
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ add_row_values as (

select
cast(dates.date_day as {{ dbt_expectations.type_datetime() }}) as date_day,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ type_float() }}) as row_value
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ dbt.type_float() }}) as row_value

from
dates
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ add_row_values as (
select
cast(dates.date_day as {{ dbt_expectations.type_datetime() }}) as date_day,
cast(dates.date_day as {{ dbt_expectations.type_timestamp() }}) as date_timestamp,
cast(groupings.generated_number as {{ type_int() }}) as group_id,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ type_float() }}) as row_value
cast(groupings.generated_number as {{ dbt.type_int() }}) as group_id,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ dbt.type_float() }}) as row_value
from
dates
cross join groupings
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ add_row_values as (

select
cast(dates.date_hour as {{ dbt_expectations.type_datetime() }}) as date_hour,
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ type_float() }}) as row_value
cast(100 * abs({{ dbt_expectations.rand() }}) as {{ dbt.type_float() }}) as row_value

from
dates
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ with data_example as (
select
1 as idx,
'2020-10-21' as date_col,
cast(0 as {{ type_float() }}) as col_numeric_a
cast(0 as {{ dbt.type_float() }}) as col_numeric_a

union all

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ set_values as (
{% for value in value_set -%}
select
{% if quote_values -%}
cast('{{ value }}' as {{ type_string() }})
cast('{{ value }}' as {{ dbt.type_string() }})
{%- else -%}
{{ value }}
{%- endif %} as value_field
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@

{% for column in columns_in_relation %}
select
cast('{{ column.name | upper }}' as {{ type_string() }}) as relation_column,
cast('{{ column.dtype | upper }}' as {{ type_string() }}) as relation_column_type
cast('{{ column.name | upper }}' as {{ dbt.type_string() }}) as relation_column,
cast('{{ column.dtype | upper }}' as {{ dbt.type_string() }}) as relation_column_type
{% if not loop.last %}union all{% endif %}
{% endfor %}
),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ set_values as (
{% for value in value_set -%}
select
{% if quote_values -%}
cast('{{ value }}' as {{ type_string() }})
cast('{{ value }}' as {{ dbt.type_string() }})
{%- else -%}
{{ value }}
{%- endif %} as value_field
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ with metric_values as (
with grouped_metric_values as (

select
{{ date_trunc(period, date_column_name) }} as metric_period,
{{ dbt.date_trunc(period, date_column_name) }} as metric_period,
{{ group_by | join(",") ~ "," if group_by }}
sum({{ column_name }}) as agg_metric_value
from
Expand Down Expand Up @@ -139,10 +139,10 @@ from
where

metric_period >= cast(
{{ dateadd(period, -test_periods, date_trunc(period, dbt_date.now())) }}
as {{ type_timestamp() }})
{{ dbt.dateadd(period, -test_periods, dbt.date_trunc(period, dbt_date.now())) }}
as {{ dbt_expectations.type_timestamp() }})
and
metric_period < {{ date_trunc(period, dbt_date.now()) }}
metric_period < {{ dbt.date_trunc(period, dbt_date.now()) }}
and

not (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@ with base_dates as (
Filtering the spine only where this remainder == 0 will return a spine with every other day as desired, i.e. [2020-01-01, 2020-01-03, 2020-01-05, ...]
#}
where mod(
cast({{ datediff("'" ~ start_date ~ "'", 'date_' ~ date_part, date_part) }} as {{ type_int() }}),
cast({{interval}} as {{ type_int() }})
cast({{ dbt.datediff("'" ~ start_date ~ "'", 'date_' ~ date_part, date_part) }} as {{ dbt.type_int() }}),
cast({{interval}} as {{ dbt.type_int() }})
) = 0
{% endif %}

Expand All @@ -68,7 +68,7 @@ model_data as (
select
{% if not interval %}

cast({{ date_trunc(date_part, date_col) }} as {{ dbt_expectations.type_datetime() }}) as date_{{ date_part }},
cast({{ dbt.date_trunc(date_part, date_col) }} as {{ dbt_expectations.type_datetime() }}) as date_{{ date_part }},

{% else %}
{#
Expand All @@ -80,13 +80,13 @@ model_data as (
subtracting that number of days from the observations will produce records [2020-01-01, 2020-01-01, 2020-01-03, 2020-01-11, 2020-01-11],
all of which align with records from the interval-date spine
#}
{{dateadd(
{{ dbt.dateadd(
date_part,
"mod(
cast(" ~ datediff("'" ~ start_date ~ "'", date_col, date_part) ~ " as " ~ type_int() ~ " ),
cast(" ~ interval ~ " as " ~ type_int() ~ " )
cast(" ~ datediff("'" ~ start_date ~ "'", date_col, date_part) ~ " as " ~ dbt.type_int() ~ " ),
cast(" ~ interval ~ " as " ~ dbt.type_int() ~ " )
) * (-1)",
"cast( " ~ date_trunc(date_part, date_col) ~ " as " ~ dbt_expectations.type_datetime() ~ ")"
"cast( " ~ dbt.date_trunc(date_part, date_col) ~ " as " ~ dbt_expectations.type_datetime() ~ ")"
)}} as date_{{ date_part }},

{% endif %}
Expand Down
2 changes: 1 addition & 1 deletion macros/schema_tests/table_shape/expect_column_to_exist.sql
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
with test_data as (

select
cast('{{ column_name }}' as {{ type_string() }}) as column_name,
cast('{{ column_name }}' as {{ dbt.type_string() }}) as column_name,
{{ matching_column_index }} as matching_column_index,
{{ column_index_matches }} as column_index_matches

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,12 @@ with latest_grouped_timestamps as (
select
{{ group_by | join(",") ~ "," if group_by }}
max(1) as join_key,
max(cast({{ timestamp_column }} as {{ type_timestamp() }})) as latest_timestamp_column
max(cast({{ timestamp_column }} as {{ dbt_expectations.type_timestamp() }})) as latest_timestamp_column
from
{{ model }}
where
-- to exclude erroneous future dates
cast({{ timestamp_column }} as {{ type_timestamp() }}) <= {{ dbt_date.now() }}
cast({{ timestamp_column }} as {{ dbt_expectations.type_timestamp() }}) <= {{ dbt_date.now() }}
{% if row_condition %}
and {{ row_condition }}
{% endif %}
Expand Down Expand Up @@ -62,8 +62,8 @@ outdated_grouped_timestamps as (
-- are the max timestamps per group older than the specified cutoff?
latest_timestamp_column <
cast(
{{ dateadd(datepart, interval * -1, dbt_date.now()) }}
as {{ type_timestamp() }}
{{ dbt.dateadd(datepart, interval * -1, dbt_date.now()) }}
as {{ dbt_expectations.type_timestamp() }}
)

),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,12 @@
{%- set default_start_date = '1970-01-01' -%}
with max_recency as (

select max(cast({{ column_name }} as {{ type_timestamp() }})) as max_timestamp
select max(cast({{ column_name }} as {{ dbt_expectations.type_timestamp() }})) as max_timestamp
from
{{ model }}
where
-- to exclude erroneous future dates
cast({{ column_name }} as {{ type_timestamp() }}) <= {{ dbt_date.now() }}
cast({{ column_name }} as {{ dbt_expectations.type_timestamp() }}) <= {{ dbt_date.now() }}
{% if row_condition %}
and {{ row_condition }}
{% endif %}
Expand All @@ -33,8 +33,8 @@ from
where
-- if the row_condition excludes all rows, we need to compare against a default date
-- to avoid false negatives
coalesce(max_timestamp, cast('{{ default_start_date }}' as {{ type_timestamp() }}))
coalesce(max_timestamp, cast('{{ default_start_date }}' as {{ dbt_expectations.type_timestamp() }}))
<
cast({{ dateadd(datepart, interval * -1, dbt_date.now()) }} as {{ type_timestamp() }})
cast({{ dbt.dateadd(datepart, interval * -1, dbt_date.now()) }} as {{ dbt_expectations.type_timestamp() }})

{% endmacro %}
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,14 @@
with relation_columns as (

{% for col_name in relation_column_names %}
select cast('{{ col_name }}' as {{ type_string() }}) as relation_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as relation_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
),
input_columns as (

{% for col_name in column_list %}
select cast('{{ col_name }}' as {{ type_string() }}) as input_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as input_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
{% for col_name in relation_column_names %}
select
{{ loop.index }} as relation_column_idx,
cast('{{ col_name }}' as {{ type_string() }}) as relation_column
cast('{{ col_name }}' as {{ dbt.type_string() }}) as relation_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
),
Expand All @@ -17,7 +17,7 @@
{% for col_name in column_list %}
select
{{ loop.index }} as input_column_idx,
cast('{{ col_name }}' as {{ type_string() }}) as input_column
cast('{{ col_name }}' as {{ dbt.type_string() }}) as input_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,14 @@
with relation_columns as (

{% for col_name in relation_column_names %}
select cast('{{ col_name }}' as {{ type_string() }}) as relation_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as relation_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
),
input_columns as (

{% for col_name in column_list %}
select cast('{{ col_name }}' as {{ type_string() }}) as input_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as input_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,14 @@
with relation_columns as (

{% for col_name in relation_column_names %}
select cast('{{ col_name }}' as {{ type_string() }}) as relation_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as relation_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
),
input_columns as (

{% for col_name in column_list %}
select cast('{{ col_name }}' as {{ type_string() }}) as input_column
select cast('{{ col_name }}' as {{ dbt.type_string() }}) as input_column
{% if not loop.last %}union all{% endif %}
{% endfor %}
)
Expand Down

0 comments on commit 8847b5a

Please sign in to comment.