From 54df20cad47cf88d45b3ddab7c686f2107c0e733 Mon Sep 17 00:00:00 2001
From: Geoff Genz <geoff@clickhouse.com>
Date: Sun, 4 Feb 2024 13:10:24 -0700
Subject: [PATCH] Revert "Bug/223 relationship test with limit (#245)"

This reverts commit d8afb93929f9c07c04bc0b3a86ead29b2e0752df.
---
 CHANGELOG.md                                  |  1 -
 dbt/adapters/clickhouse/impl.py               | 19 ++----------
 .../macros/schema_tests/relationships.sql     |  1 -
 dbt/include/clickhouse/macros/utils/utils.sql | 30 -------------------
 4 files changed, 2 insertions(+), 49 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 6e5a1057..1cbef238 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,7 +2,6 @@
 #### Bug Fix
 - Fixed an issue where Materialize Views would break with a custom schema.  Thanks to [Rory Sawyer](https://github.com/SoryRawyer)
 for the PR!
-- A few tests with LIMIT clause were broken due to parsing error when having settings in the query ([issue](https://github.com/ClickHouse/dbt-clickhouse/issues/223)). We added a dedicated limit placer, that takes into account the settings section (using a comment flag `-- settings_section` within the query).
 
 ### Release [1.7.1], 2023-12-13
 #### Bug Fixes
diff --git a/dbt/adapters/clickhouse/impl.py b/dbt/adapters/clickhouse/impl.py
index 09047502..f5b9b0cf 100644
--- a/dbt/adapters/clickhouse/impl.py
+++ b/dbt/adapters/clickhouse/impl.py
@@ -412,14 +412,7 @@ def get_model_settings(self, model):
         res = []
         for key in settings:
             res.append(f' {key}={settings[key]}')
-        if len(res) == 0:
-            return ''
-        else:
-            settings_str = 'SETTINGS ' + ', '.join(res) + '\n'
-            return f"""
-                        -- end_of_sql
-                        {settings_str}
-                        """
+        return '' if len(res) == 0 else 'SETTINGS ' + ', '.join(res) + '\n'
 
     @available
     def get_model_query_settings(self, model):
@@ -427,15 +420,7 @@ def get_model_query_settings(self, model):
         res = []
         for key in settings:
             res.append(f' {key}={settings[key]}')
-
-        if len(res) == 0:
-            return ''
-        else:
-            settings_str = 'SETTINGS ' + ', '.join(res) + '\n'
-            return f"""
-            -- settings_section
-            {settings_str}
-            """
+        return '' if len(res) == 0 else 'SETTINGS ' + ', '.join(res) + '\n'
 
     @available.parse_none
     def get_column_schema_from_query(self, sql: str, *_) -> List[ClickHouseColumn]:
diff --git a/dbt/include/clickhouse/macros/schema_tests/relationships.sql b/dbt/include/clickhouse/macros/schema_tests/relationships.sql
index f602fecc..b756a99c 100644
--- a/dbt/include/clickhouse/macros/schema_tests/relationships.sql
+++ b/dbt/include/clickhouse/macros/schema_tests/relationships.sql
@@ -19,7 +19,6 @@ left join parent
     on child.from_field = parent.to_field
 
 where parent.to_field is null
--- end_of_sql
 settings join_use_nulls = 1
 
 {% endmacro %}
diff --git a/dbt/include/clickhouse/macros/utils/utils.sql b/dbt/include/clickhouse/macros/utils/utils.sql
index d81e6da3..9eb391f9 100644
--- a/dbt/include/clickhouse/macros/utils/utils.sql
+++ b/dbt/include/clickhouse/macros/utils/utils.sql
@@ -1,33 +1,3 @@
-{% macro clickhouse__get_test_sql(main_sql, fail_calc, warn_if, error_if, limit) -%}
-    {% set main_sql_formatted = clickhouse__place_limit(main_sql, limit) if limit !=None else main_sql%}
-    select
-      {{ fail_calc }} as failures,
-      {{ fail_calc }} {{ warn_if }} as should_warn,
-      {{ fail_calc }} {{ error_if }} as should_error
-    from (
-      {{ main_sql_formatted }}
-    ) dbt_internal_test
-
-{%- endmacro %}
-
-
--- This macro is designed to add a LIMIT clause to a ClickHouse SQL query while preserving any ClickHouse settings specified in the query.
--- When multiple queries are nested, the limit will be attached to the outer query
-{% macro clickhouse__place_limit(query, limit) -%}
-   {% if 'settings' in query.lower()%}
-        {% if '-- end_of_sql' not in query.lower()%}
-            {{exceptions.raise_compiler_error("-- end_of_sql must be set when using ClickHouse settings")}}
-        {% endif %}
-        {% set split_by_settings_sections = query.split("-- end_of_sql")%}
-        {% set split_by_settings_sections_with_limit = split_by_settings_sections[-2] + "\n LIMIT " + limit|string  + "\n" %}
-        {% set query_with_limit = "-- end_of_sql".join(split_by_settings_sections[:-2] + [split_by_settings_sections_with_limit, split_by_settings_sections[-1]])%}
-        {{query_with_limit}}
-    {% else %}
-    {{query}}
-    {{"limit " ~ limit}}
-    {% endif %}
-{%- endmacro %}
-
 {% macro clickhouse__any_value(expression) -%}
     any({{ expression }})
 {%- endmacro %}