informatikr / hedis

A Redis client library for Haskell.
http://hackage.haskell.org/package/hedis
BSD 3-Clause "New" or "Revised" License
327 stars 121 forks source link

We need internals of Connection, or an `isClusteredConn` function #216

Closed jberryman closed 9 months ago

jberryman commented 9 months ago

Just trying out HEAD for the 9.6 support, and I don't think the API supports some existing code we have of the form:

case redisConn of
      Redis.NonClusteredConnection _ -> ...
      Redis.ClusteredConnection _ _ -> ...
jberryman commented 9 months ago

sorry, I realized we're using a fork of hedis; I'm not sure this is relevant

qnikst commented 6 months ago

@jberryman actually I would more than happy in case if cluster implementation would be ported upstream, I'm not sure that there is any reason to keep two separate forks each with their own flaws. Unfortunately I'll not be able to maintain server part myself as we are using sentinel based approach, but I would appreciate all PR-s from the fork