search

plausible / ecto_ch

Ecto ClickHouse adapter
MIT License
72 stars 10 forks source link

field options in Ch ecto type #52

Closed ruslandoga closed 1 year ago

ruslandoga commented 1 year ago

this doesn't work, find out why and open a PR into Ecto to support field options in parameterised Ecto types

_Originally posted by @ruslandoga in https://github.com/plausible/chto/pull/50#discussion_r1174904055_

ruslandoga commented 1 year ago

field :a, Ch, type: "Array(T)" can't be used everywhere {:array, _} is used. For example, it raises in Ecto.Query.Planner.normalize_param for queries like where: 0 in t.ints

This should be fixed in #67

ruslandoga commented 1 year ago

Both field options and array seem to be working now.