Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace redis client #3968

Closed
lengrongfu opened this issue Jul 25, 2023 · 2 comments · Fixed by #4019
Closed

Replace redis client #3968

lengrongfu opened this issue Jul 25, 2023 · 2 comments · Fixed by #4019

Comments

@lengrongfu
Copy link

current use github.com/gomodule/redigo/redis client, but it only support single node redis , i hope add redis sentinel, ref pr this #2886, we can use https://github.com/redis/go-redis client to use redis cluster、redis sentinel、single redis server.

@lengrongfu
Copy link
Author

@dmcgowan What do you think of the proposal to replace the golang redis client?

@davidspek
Copy link
Collaborator

@lengrongfu I think it makes sense to use what seems to be the official redis client. Although we might want to double check the BSD 2-Clause is compatible with the Apache2 license of this repo (I believe it is).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants