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

Expose RedisClient.connect(RedisCodec, RedisURI) and RedisClient.connectAsync(RedisCodec, RedisURI) #108

Closed
mp911de opened this issue Jul 30, 2015 · 1 comment
Labels
type: feature A new feature
Milestone

Comments

@mp911de
Copy link
Collaborator

mp911de commented Jul 30, 2015

RedisClient is a heavy-weight resource that encapsulates thread pools etc. Users working with multiple hosts (such as wrappers around lettuce) are interested in reusing the RedisClient to not exhaust the JVM resources.

The methods are currently private and should be made public.

Reusing the RedisClusterClient for different connection points plays as well into this issue but should be addressed in a later issue.

@mp911de mp911de added the type: feature A new feature label Jul 30, 2015
@mp911de mp911de added this to the Lettuce 3.3 milestone Jul 30, 2015
mp911de added a commit that referenced this issue Jul 31, 2015
@mp911de
Copy link
Collaborator Author

mp911de commented Jul 31, 2015

Implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature A new feature
Projects
None yet
Development

No branches or pull requests

1 participant