This repository has been archived by the owner. It is now read-only.

Redis URL is invalid #5

Closed
mperham opened this Issue Mar 31, 2013 · 2 comments

Comments

Projects
None yet
2 participants
@mperham

mperham commented Mar 31, 2013

In the sidekiq blog post, your Redis URLs are of the form: "redis://#{$redis_config[:host]}:#{$redis_config[:port]}/sidekiq"

This is actually invalid: redis expects the "path" part of the url to be a number from 0 to 15, indicating which DB to use. Anything that is not a number uses DB 0. In the URL above "sidekiq" is meaningless and Redis will just ignore it. It would be a good idea to either remove it or change it to be a number.

@drnic

This comment has been minimized.

Show comment Hide comment
@drnic

drnic Mar 31, 2013

Member

Ahh thanks. I'm sure I've copied this from somewhere and thought it needed to be the same as the namespace label.

Do the 16 databases that come with one redis server have any community significance? Does sidekiq like to use a particular number?

Member

drnic commented Mar 31, 2013

Ahh thanks. I'm sure I've copied this from somewhere and thought it needed to be the same as the namespace label.

Do the 16 databases that come with one redis server have any community significance? Does sidekiq like to use a particular number?

@mperham

This comment has been minimized.

Show comment Hide comment
@mperham

mperham Mar 31, 2013

Yeah, everyone copies the URL pattern and it happens to work because of to_i.

Nope, no standard. Not many people use multiple databases, most just use namespacing.

On 31 Mar 2013, at 07:23, Dr Nic Williams notifications@github.com wrote:

Ahh thanks. I'm sure I've copied this from somewhere and thought it needed to be the same as the namespace label.

Do the 16 databases that come with one redis server have any community significance? Does sidekiq like to use a particular number?


Reply to this email directly or view it on GitHub.

mperham commented Mar 31, 2013

Yeah, everyone copies the URL pattern and it happens to work because of to_i.

Nope, no standard. Not many people use multiple databases, most just use namespacing.

On 31 Mar 2013, at 07:23, Dr Nic Williams notifications@github.com wrote:

Ahh thanks. I'm sure I've copied this from somewhere and thought it needed to be the same as the namespace label.

Do the 16 databases that come with one redis server have any community significance? Does sidekiq like to use a particular number?


Reply to this email directly or view it on GitHub.

@drnic drnic referenced this issue in drnic/appscrolls Mar 31, 2013

Closed

redis url ends with a number not a string #43

@drnic drnic closed this in 58f8456 Mar 31, 2013

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