-
Notifications
You must be signed in to change notification settings - Fork 5
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
access to RedisClusterClient/RedisClient #51
Comments
Hi @jmhodges! Thanks for the kind words! 😃 When in the lifecycle of the client setup would you need access to the |
Oh, lemme narrow the focus to RedisClient and let's ignore the "connection pooling" stuff, because I was still learning a legacy codebase this is going in: We want to get access to pubsub stuff with |
@jmhodges That seems totally reasonable! One way this can be easily addressed is by adding a protected method that will allow you to extend One follow-up question: do you simply need to configure additional things on the |
Not sure if directly related, but I too was hoping to use this for Pub/Sub. |
I was trying to use this wonderful library but the app I'm working on needs access to the underlying RedisClusterClient in order to handle connection pooling. I can see that there's lots of useful things done with the StatefulRedisClusterConnection before it's returned, so I understand that this is possibly outside the scope of the project.
But it would be nice to get all this great JSON config work while will getting the Client I needed.
The text was updated successfully, but these errors were encountered: