Merge pull request #232 from microsoft/backport_1.7_230

Backport 1.7 230
microsoft · Apr 8, 2024 · 6a5a4f5 · 6a5a4f5
2 parents f69cb95 + afa0232
commit 6a5a4f5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,14 @@
 # Changelog
+
+## v1.7.2
+
+### Under the hood
+
+* Backport #230 which disables `dbt show`'s `--limit` argument.
+
 ## v1.7.1
 
-#### Under the hood
+### Under the hood
 
  * Allow connection to synapse using `authentication: sql` and `user: sqladminuser` in `profiles.yml`
  - see ([#218](https:/microsoft/dbt-synapse/issues/218))

diff --git a/dbt/include/synapse/macros/adapters/show.sql b/dbt/include/synapse/macros/adapters/show.sql
@@ -3,10 +3,11 @@
  {{ sql_header }}
  {%- endif -%}
  {%- if limit is not none -%}
- {{ get_limit_subquery_sql(compiled_code, limit) }}
- {%- else -%}
- {{ compiled_code }}
+ {%- set warn = "--limit is ignored. Synapse doesn't support the implementation" -%}
+ {{ log(warn, info=True) }}
  {%- endif -%}
+ {{ compiled_code }}
+
 {% endmacro %}
 
 {% macro get_limit_subquery_sql(sql, limit) %}