bugfix/databricks-sql-path-adjustment (#123)

* bugfix/databricks-sql-path-adjustment * docs regen
fivetran · Apr 9, 2024 · d355614 · d355614
1 parent 4ada430
commit d355614
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,10 @@
+# dbt_fivetran_log v1.7.2
+[PR #123](https://github.com/fivetran/dbt_fivetran_log/pull/123) includes the following updates:
+
+## Bug Fixes
+- Removal of the leading `/` from the `target.http_path` regex search within the `is_databricks_sql_warehouse()` macro to accurately identify SQL Warehouse Databricks destinations in Quickstart.
+  - The macro above initially worked as expected in dbt core environments; however, in Quickstart implementations this data model was not working. This was due to Quickstart removing the leading `/` from the `target.http_path`. Thus resulting in the regex search to always fail. 
+
 # dbt_fivetran_log v1.7.1
 [PR #121](https://github.com/fivetran/dbt_fivetran_log/pull/121) includes the following updates:
 

diff --git a/dbt_project.yml b/dbt_project.yml
@@ -1,6 +1,6 @@
 config-version: 2
 name: 'fivetran_log'
-version: '1.7.1'
+version: '1.7.2'
 require-dbt-version: [">=1.3.0", "<2.0.0"]
 
 models:

diff --git a/docs/catalog.json b/docs/catalog.json
diff --git a/docs/manifest.json b/docs/manifest.json
diff --git a/docs/run_results.json b/docs/run_results.json
diff --git a/integration_tests/dbt_project.yml b/integration_tests/dbt_project.yml
@@ -1,5 +1,5 @@
 name: 'fivetran_log_integration_tests'
-version: '1.7.1'
+version: '1.7.2'
 
 config-version: 2
 profile: 'integration_tests'

diff --git a/macros/is_databricks_sql_warehouse.sql b/macros/is_databricks_sql_warehouse.sql
@@ -2,7 +2,7 @@
     {% if target.type in ('databricks') %}
         {% set re = modules.re %}
         {% set path_match = target.http_path %}
-        {% set regex_pattern = "/sql/.+/warehouses/" %}
+        {% set regex_pattern = "sql/.+/warehouses/" %}
         {% set match_result = re.search(regex_pattern, path_match) %}
         {% if match_result %}
             {{ return(True) }}