Skip to content

Commit 9046153

Browse files
authored
Merge pull request #1 from lanalabs/fix_stringification_of_arrays
FIX unnecessary stringification of list values
2 parents c5eb974 + 8b37b4b commit 9046153

File tree

4 files changed

+4
-44
lines changed

4 files changed

+4
-44
lines changed

lib/clickhousex/http_client.ex

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
defmodule Clickhousex.HTTPClient do
22
@moduledoc false
33

4-
alias Clickhousex.Types
5-
64
@selected_queries_regex ~r/^(SELECT|SHOW|DESCRIBE|EXISTS)/i
75
@req_headers [{"Content-Type", "text/plain"}]
86

@@ -36,10 +34,9 @@ defmodule Clickhousex.HTTPClient do
3634
data
3735
|> Enum.map(fn data_row ->
3836
meta
39-
|> Enum.map(fn %{"name" => column, "type" => column_type} ->
40-
Types.decode(data_row[column], column_type)
37+
|> Enum.map(fn %{"name" => column} ->
38+
data_row[column]
4139
end)
42-
|> List.to_tuple()
4340
end)
4441

4542
{command, columns, rows}

lib/clickhousex/query.ex

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -38,24 +38,7 @@ defimpl DBConnection.Query, for: Clickhousex.Query do
3838
def decode(_query, result, _opts) do
3939
case result.command do
4040
:selected ->
41-
rows = result.rows
42-
43-
new_rows =
44-
Enum.map(rows, fn el ->
45-
list1 = Tuple.to_list(el)
46-
47-
Enum.map(list1, fn el1 ->
48-
cond do
49-
is_list(el1) ->
50-
to_string(el1)
51-
52-
true ->
53-
el1
54-
end
55-
end)
56-
end)
57-
58-
Map.put(result, :rows, new_rows)
41+
Map.put(result, :rows, result.rows)
5942

6043
_ ->
6144
result

lib/clickhousex/types.ex

Lines changed: 0 additions & 20 deletions
This file was deleted.

mix.exs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ defmodule Clickhousex.Mixfile do
44
def project do
55
[
66
app: :clickhousex,
7-
version: "0.2.4",
7+
version: "0.2.5",
88
elixir: "~> 1.5",
99
deps: deps(),
1010
package: package(),

0 commit comments

Comments
 (0)