Infering array types

[aerosol]

Reference: plausible/analytics#6049 (comment)

TODO to address:

ecto_ch/lib/ecto/adapters/clickhouse/connection.ex

Line 1234 in 9a517fe

# TODO check whole list

Will reference this ticket in brute force fix on plausible end for now.

[aerosol]

@ruslandoga any joy with this? I think the fix you suggested should work as-is, no? If so I can submit it if you like.

[ruslandoga]

Sorry for the delay! I've published v0.8.5 with the fix. I'll check what ecto_ch version Plausible is using right now and if there are breaking changes I'll also backport it.

---

UPDATE: oh, it's on v0.8.4 :) I think it's safe to upgrade then!

[aerosol]

Thank you @ruslandoga, nice work

[aerosol]

Hm after upgrade we're getting

     ** (Ch.Error) Code: 48. DB::Exception: Serialization is not implemented for type Nothing: val
ue [[]] cannot be parsed as Array(Nothing) for query parameter '$4'. (NOT_IMPLEMENTED) (version 26
.1.2.11 (official build))

https://github.com/plausible/analytics/actions/runs/21912396119/job/63269829920?pr=6064

[ruslandoga]

[[]] should probably be Array(Array(Nothing)) instead of just Array(Nothing).

There was a mistake in the recursive step of param_type/1. I'll fix it in #264, retire v0.8.5, and release v0.8.6.

---

UPDATE: I've released v0.8.6 with another fix.

[aerosol]

Thanks, this works!