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

Unexpected 'Invalid database number' exception for database > 15 #420

Closed
ChitraGuru opened this issue Dec 7, 2016 · 2 comments
Closed
Labels
type: bug A general bug
Milestone

Comments

@ChitraGuru
Copy link

Lettuce Version: 4.2.2.Final

When trying to configure RedisURI, and specify the database, there is a check for the value being <= 15 (https://github.com/mp911de/lettuce/blob/master/src/main/java/com/lambdaworks/redis/RedisURI.java#L1144)

However, it is possible to configure redis to allow more than 16 databases. (http://download.redis.io/redis-stable/redis.conf)

# Set the number of databases. The default database is DB 0, you can select
# a different one on a per-connection basis using SELECT <dbid> where
# dbid is a number between 0 and 'databases'-1
databases 16

Is there any other reason to disallow databases >15 here?

Thanks.

@mp911de mp911de added the type: bug A general bug label Dec 7, 2016
@mp911de
Copy link
Collaborator

mp911de commented Dec 7, 2016

Good catch. That's a bug.

@mp911de
Copy link
Collaborator

mp911de commented Dec 7, 2016

That's fixed now.

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

No branches or pull requests

2 participants