Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Vladimir Kostyukov
    @vkostyukov
    @darri89 Looks your right; finagle/finagle-postgres#117 is failng.
    Leon Maia
    @leonmaia
    Hi @darri89 thanks for the heads up. I'll have a look on it.
    Leon Maia
    @leonmaia
    @darri89 I got a green build with 19.2.0, I'll be doing a new release tomorrow.
    darri89
    @darri89
    @leonmaia great thanks!
    Yarek T
    @YarekTyshchenko
    Hi. I'm trying to write a query that uses a WHERE (a, b) IN ((1, 2), (3, 4)) style of where clause. It looks like it's not possible to pass a Seq[T] as a param. Is there any way around this?
    Yarek T
    @YarekTyshchenko
    Interestingly using sql"<query>" syntax works when passing a list of a type fine, but doesn't work with a Tuple, or a case class that has the two fields
    can you post your query and the error(s) you get? Sorry, it’s been a while since I wrote that stuff and it’s not well documented :( There are some quirks of the query DSL that you can hopefully see in that spec
    Yarek T
    @YarekTyshchenko

    This test should demonstrate the problem:

          "Seq of tuple params for IN operator" in {
            val p1 = Seq(("foo", "bar"), ("three", "four"))
            expectQuery("SELECT * FROM foo WHERE (a, b) IN (($1, $2), ($3, $4))", "foo", "bar", "three", "four") {
              sql"SELECT * FROM foo WHERE a IN ($p1)"
            }
          }

    Its a bit of a weird syntax, but postgres does support it

    @jeremyrsmith ^^.
    Philippe Laflamme
    @plaflamme
    Hi, sorry for the noise, but I'm looking for feedback on finagle/finagle-postgres#128 I'd just like to know if it's something that will be considered for merging before adding more commits...
    Philippe Laflamme
    @plaflamme
    Bump ^
    jeremyrsmith
    @jeremyrsmith
    @plaflamme I think it’s a good direction. I’d merge it if it were fleshed out a bit; I’d just try to minimize API breakage if possible.
    Philippe Laflamme
    @plaflamme
    Great, thanks for the feedback! It's been a while, so what parts of it would need to be more "fleshed out"? Happy to do those first. I'll see what I can do for the API breakage...
    Philippe Laflamme
    @plaflamme
    @jeremyrsmith I've added the remaining bits and also undone the API breakage. Please take a look finagle/finagle-postgres#128
    Philippe Laflamme
    @plaflamme
    @dangerousben @leonmaia Any chance we can get a 0.12.0 release with the latest changes (https://github.com/finagle/finagle-postgres/compare/v0.11.0...master)?
    Ben Spencer
    @dangerousben
    I don't have access to cut a release (afaik), if @leonmaia's not around then @vkostyukov might
    jeremyrsmith
    @jeremyrsmith
    I have access but TBH I’m not sure how it works anymore. I used to basically just sbt +publish but it uses a more complicated (undoubtedly better) process now. Maybe linked to github releases?
    Leon Maia
    @leonmaia
    For generating a new release you just need to update version.sbt with the new release number removing the snapshot tag. The rest is done automagically :)
    Sorry - I'm not using postgres at work, moved to mysql and been very busy for the last year working with Kafka. So I'm not with the time to be 'very' active here.
    I'll merge the update PRs from scala-stewart and tag a new release.
    Leon Maia
    @leonmaia
    v0.12.0 has been published @plaflamme, thanks a lot for the contribution! Release notes: https://github.com/finagle/finagle-postgres/releases/tag/v0.12.0
    Philippe Laflamme
    @plaflamme
    Thanks so much! BTW: the Kafka work you are referring to, is that related to Finagle? Being curious, because we have a finagle-kafka client and are wondering if people are willing to contirbute
    Leon Maia
    @leonmaia
    I'll DM you so we don't pollute here too much with off-topic :)
    Ben Spencer
    @dangerousben
    thanks @leonmaia! (and @plaflamme for the code)