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.

Sign up for GitHub

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

Jump to bottom

Allow the developer to change the Jedis pool parameters #3

Open

exabrial opened this issue Jul 8, 2023 · 0 comments

Owner

exabrial commented Jul 8, 2023

Right now the Jedis pool parameters are hard coded here: https://github.com/exabrial/redex-sm/blob/master/src/main/java/com/github/exabrial/redexsm/jedis/JedisRedisService.java#L75

While these settings should work well for the vast majority of cases, they should be customizable.

Constraints

The sessionManager is ignorant of the Redis library being used. It should probably stay that way in case we need to switch from Jedis. So avoid any Jedis specific settings.
One way to accomplish that may be to include the parameters as part of the Redis URL (Sort of like how MySQL does)

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