_ and $ can be in identifiers

plausible · Sep 6, 2024 · 0415d9f · 0415d9f
1 parent efd6d2d
commit 0415d9f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/ch/types.ex b/lib/ch/types.ex
@@ -481,7 +481,7 @@ defmodule Ch.Types do
   defguardp is_numeric(char) when char >= ?0 and char <= ?9
 
   defp decode_identifier(<<a, rest::bytes>>, len, original, stack, acc)
-       when is_alpha(a) or is_numeric(a) do
+       when is_alpha(a) or is_numeric(a) or a == ?_ or a == ?$ do
     decode_identifier(rest, len + 1, original, stack, acc)
   end
 

diff --git a/test/ch/query_test.exs b/test/ch/query_test.exs
@@ -102,6 +102,8 @@ defmodule Ch.QueryTest do
     test "decode tuples", %{conn: conn} do
       assert [[{"Hello", 123}]] = Ch.query!(conn, "select ('Hello', 123)").rows
       assert [[{"Hello", 123}]] = Ch.query!(conn, "select ('Hello' as a, 123 as b)").rows
+      assert [[{"Hello", 123}]] = Ch.query!(conn, "select ('Hello' as a_, 123 as b)").rows
+      assert [[{"Hello", 123}]] = Ch.query!(conn, "select ('Hello' as a$, 123 as b)").rows
     end
 
     test "decode network types", %{conn: conn} do