diff --git a/CHANGELOG.md b/CHANGELOG.md index df6704cc..75833b7b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,13 +1,24 @@ # dbt-utils v0.8.0 ## 🚨 Breaking changes +- dbt ONE POINT OH is here! This version of dbt-utils requires `dbt-core ~=1.0.0` (any minor and patch version of v1), which means far less need for compatibility releases in the future. - The partition column in the `mutually_exclusive_ranges` test is now always called `partition_by_col`. This enables compatibility with `--store-failures` when multiple columns are concatenated together. If you have models built on top of the failures table, update them to reflect the new column name. ([#423](https://github.com/dbt-labs/dbt-utils/issues/423), [#430](https://github.com/dbt-labs/dbt-utils/pull/430)) ## Contributors: - [codigo-ergo-sum](https://github.com/codigo-ergo-sum) (#430) -# dbt-utils v0.7.4 +# dbt-utils 0.7.5 🚨 This is a compatibility release in preparation for `dbt-core` v1.0.0 (🎉). Projects using dbt-utils 0.7.4 with dbt-core v1.0.0 can expect to see a deprecation warning. This will be resolved in dbt_utils v0.8.0. +## Fixes +- Regression in `get_column_values()` where the default would not be respected if the model didn't exist. ([#444](https://github.com/dbt-labs/dbt-utils/issues/444), [#448](https://github.com/dbt-labs/dbt-utils/pull/448)) + +## Under the hood +- get_url_host() macro now correctly handles URLs beginning with android-app:// (#426) + +## Contributors: +- [foundinblank](https://github.com/foundinblank) + +# dbt-utils v0.7.4 ## Fixes - `get_column_values()` now works correctly with mixed-quoting styles on Snowflake ([#424](https://github.com/dbt-labs/dbt-utils/issues/424), [#440](https://github.com/dbt-labs/dbt-utils/pull/440)) - Remove extra semicolon in `insert_by_period` materialization that was causing errors ([#439](https://github.com/dbt-labs/dbt-utils/pull/439)) diff --git a/integration_tests/data/sql/data_get_column_values_dropped.csv b/integration_tests/data/sql/data_get_column_values_dropped.csv new file mode 100644 index 00000000..5651149b --- /dev/null +++ b/integration_tests/data/sql/data_get_column_values_dropped.csv @@ -0,0 +1,12 @@ +field +a +b +c +d +e +f +g +g +g +g +g diff --git a/integration_tests/data/web/data_url_host.csv b/integration_tests/data/web/data_url_host.csv index 36daaa00..2350bb7a 100644 --- a/integration_tests/data/web/data_url_host.csv +++ b/integration_tests/data/web/data_url_host.csv @@ -4,4 +4,4 @@ http://witanddelight.com/2018/01/tips-tricks-how-run-half-marathon-first-time/,w https://www.nytimes.com/2018/01/01/blog,www.nytimes.com android-app://m.facebook.com/,m.facebook.com docs.nytimes.com/2021/01/01/index.js?utm_source=google,docs.nytimes.com -https://m.facebook.com/,m.facebook.com +https://m.facebook.com/,m.facebook.com \ No newline at end of file diff --git a/integration_tests/dbt_project.yml b/integration_tests/dbt_project.yml index 40db9050..9631a8c4 100644 --- a/integration_tests/dbt_project.yml +++ b/integration_tests/dbt_project.yml @@ -54,6 +54,12 @@ seeds: sql: data_events_20180103: +schema: events + + data_get_column_values_dropped: + # TODO: Ideally this would use the adapter, but it gives a "Tried to drop relation, but its type is null" error. + #+post-hook: "{% do adapter.drop_relation(this) %}" + +post-hook: "drop table {{ this }}" + schema_tests: data_test_sequential_timestamps: diff --git a/integration_tests/tests/sql/test_get_column_values_use_default.sql b/integration_tests/tests/sql/test_get_column_values_use_default.sql new file mode 100644 index 00000000..7f9fa3a4 --- /dev/null +++ b/integration_tests/tests/sql/test_get_column_values_use_default.sql @@ -0,0 +1,26 @@ + +{% set column_values = dbt_utils.get_column_values(ref('data_get_column_values_dropped'), 'field', default=['y', 'z'], order_by="field") %} + +with expected as ( + select {{ dbt_utils.safe_cast("'y'", dbt_utils.type_string()) }} as expected_column_value union all + select {{ dbt_utils.safe_cast("'z'", dbt_utils.type_string()) }} as expected_column_value +), + +actual as ( + + {% for val in column_values %} + select {{ dbt_utils.safe_cast("'" ~ val ~ "'", dbt_utils.type_string()) }} as actual_column_value + {% if not loop.last %} + union all + {% endif %} + {% endfor %} +), + +failures as ( + select * from actual + where actual.actual_column_value not in ( + select expected.expected_column_value from expected + ) +) + +select * from failures \ No newline at end of file diff --git a/macros/sql/get_column_values.sql b/macros/sql/get_column_values.sql index bc05fb07..57b150a6 100644 --- a/macros/sql/get_column_values.sql +++ b/macros/sql/get_column_values.sql @@ -15,15 +15,18 @@ {# TODO: Change the method signature in a future 0.x.0 release #} {%- set target_relation = table -%} + {# adapter.load_relation is a convenience wrapper to avoid building a Relation when we already have one #} + {% set relation_exists = (load_relation(target_relation)) is not none %} + {%- call statement('get_column_values', fetch_result=true) %} - {%- if not target_relation and default is none -%} + {%- if not relation_exists and default is none -%} - {{ exceptions.raise_compiler_error("In get_column_values(): relation " ~ table ~ " does not exist and no default value was provided.") }} + {{ exceptions.raise_compiler_error("In get_column_values(): relation " ~ target_relation ~ " does not exist and no default value was provided.") }} - {%- elif not target_relation and default is not none -%} + {%- elif not relation_exists and default is not none -%} - {{ log("Relation " ~ table ~ " does not exist. Returning the default value: " ~ default) }} + {{ log("Relation " ~ target_relation ~ " does not exist. Returning the default value: " ~ default) }} {{ return(default) }}