Merge pull request #338 from 3fonov/main

Fix quotation in split_part macro
ClickHouse · Oct 30, 2024 · 6e92d04 · 6e92d04
2 parents 51bf0e1 + efec997
commit 6e92d04
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/dbt/include/clickhouse/macros/utils/utils.sql b/dbt/include/clickhouse/macros/utils/utils.sql
@@ -54,7 +54,7 @@
 
 
 {% macro clickhouse__split_part(string_text, delimiter_text, part_number) %}
-    splitByChar('{{delimiter_text}}', {{ string_text }})[{{ part_number }}]
+    splitByString({{delimiter_text}}, {{ string_text }})[{{ part_number }}]
 {% endmacro %}
 
 
@@ -108,3 +108,4 @@
 {% macro clickhouse__array_concat(array_1, array_2) -%}
    arrayConcat({{ array_1 }}, {{ array_2 }})
 {% endmacro %}
+
diff --git a/tests/integration/adapter/utils/test_split_part.py b/tests/integration/adapter/utils/test_split_part.py
@@ -10,23 +10,23 @@
 )
 
 select
-    {{ split_part('parts', '|', 1) }} as actual,
+    {{ split_part('parts', "'|'", 1) }} as actual,
     result_1 as expected
 
 from data
 
 union all
 
 select
-    {{ split_part('parts', '|', 2) }} as actual,
+    {{ split_part('parts', "'|'", 2) }} as actual,
     result_2 as expected
 
 from data
 
 union all
 
 select
-    {{ split_part('parts', '|', 3) }} as actual,
+    {{ split_part('parts', "'|'", 3) }} as actual,
     result_3 as expected
 
 from data