ecto_ch icon indicating copy to clipboard operation
ecto_ch copied to clipboard

to_inline_sql/2 doesn't map large numbers to the correct type

Open oceanlewis opened this issue 6 months ago • 1 comments

There was recently an addition of some new cases for Ecto.Adapters.ClickHouse.Connection.param_type/1 to handle mapping numbers to Int128/256/etc. This works great when doing the usual Repo.all calls, but calling Ecto.Adapters.ClickHouse.to_inline_sql/2 seems to miss this codepath and won't cast parameterized numbers correctly.

oceanlewis avatar Aug 08 '24 20:08 oceanlewis