dbt-labs · joellabes · Dec 2, 2021 · Nov 11, 2021 · Nov 26, 2021 · Dec 2, 2021
diff --git 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
@@ -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
@@ -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
diff --git 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
@@ -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
diff --git 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) }}
-Original file line number
+Diff line change
@@ -0,0 +1,12 @@
+    field
+    a
+    b
+    c
+    d
+    e
+    f
+    g
+    g
+    g
+    g
+    g