Merge pull request #205 from bjgbeelen/add-spark-identifier

Update star macro to use adapter.quote
dbt-labs · Apr 23, 2020 · 7abce81 · 7abce81
2 parents 99d44a3 + c95ed6b
commit 7abce81
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/macros/cross_db_utils/identifer.sql b/macros/cross_db_utils/identifer.sql
diff --git a/macros/cross_db_utils/identifier.sql b/macros/cross_db_utils/identifier.sql
@@ -0,0 +1,12 @@
+{% macro identifier(value) %} 
+ {% do exceptions.warn("Warning: the `identifier` macro is no longer supported and will be deprecated in a future release of dbt-utils. Use `adapter.quote` instead") %}
+ {{ adapter_macro('dbt_utils.identifier', value) }} 
+{% endmacro %} 
+
+{% macro default__identifier(value) -%} 
+ "{{ value }}" 
+{%- endmacro %} 
+
+{% macro bigquery__identifier(value) -%} 
+ `{{ value }}` 
+{%- endmacro %}
diff --git a/macros/sql/star.sql b/macros/sql/star.sql
@@ -19,7 +19,7 @@
 
  {%- for col in include_cols %}
 
- {%- if relation_alias %}{{ relation_alias }}.{% else %}{%- endif -%}{{ dbt_utils.identifier(col)|trim }}
+ {%- if relation_alias %}{{ relation_alias }}.{% else %}{%- endif -%}{{ adapter.quote(col)|trim }}
  {%- if not loop.last %},{{ '\n ' }}{% endif %}
 
  {%- endfor -%}