Problem connecting with Redis Cluster

[carlogilmar]

Describe the bug

Recently we're trying to connect a redis cluster for use with Hammer, but we got an error that comes from Redix:

Following this: whatyouhide/redix#240, looks like this error comes from Redix that doesn't have support for the redis cluster moved errors.

Not sure what is the best way to handle this, I think we can handle the errors coming from Redix and create a new connection for the correct node and using redix we can send and get the data.

[epinault]

oh interesting. I did not know also that was a thing .Thanks for linking the original issue. So what fixes it? I am unclear what moves does? seems that we would need to restart the pool to fix it?

[carlogilmar]

I'm not sure how a redis cluster works, but looks like when you add a new record, it will stored in a certain connection, and if you need a record stored there, you need to execute the command in that particular connection:

But, nevermind, I'm still looking into this 👀

[carlogilmar]

But if I added the records using the redis-cli, looks like you can add and get the data from any part of the cluster 🤔

[carlogilmar]

Happy to help to add changes for this @epinault 😁

I added some changes here: #40

[epinault]

left some question in the MR. Happy to look at it more next week.

[sonic182]

@epinault I think it is easier if eredis_cluster can be used for redis_cluster case

and for normal redis, the eredis package

Basically, to change the Redix dependency to eredis + eredis_cluster in favor of more features

[epinault]

yea but that package is not getting a lot maintenance. I dont see any releases in 2 years but did notice a merge like 7 months ago. So I am worried about how maintained that library is. But we could create a configurable backend maybe so you can pick between the best of both world when needed? Redix, Eredis or Eredis cluster as possible backend? Though I am not sure how the eredis cluster works to be honest and would need to see how it fits . Otherwise we could create a special backend in a different package

[epinault]

i have release 7.x that has much better performances at this time compare to the old release.