zio / zio-jdbc

A small, idiomatic ZIO interface to JDBC.
Apache License 2.0
82 stars 64 forks source link

remove `Setter.arraySetter` #180

Closed oyvindberg closed 1 year ago

oyvindberg commented 1 year ago

I want to interpolate in postgres arrays as parameters - not explode them as multiple parameters. It's basically impossible to implement support for Array columns with this implicit present.

Other libraries like anorm and doobie have settled on this distinction already, scala sequences are exploded in the sql and Arrays are passed right through (with the right typeclass instances, of course). I suggest we do the same for zio-jdbc.

This is of course a breaking change, but probably doesn't affect many people. The workaround will be to say myArray.toList or similar

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.