Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Redis syntax error in handbook #39

Closed
priscillahuang opened this Issue Aug 11, 2012 · 4 comments

Comments

Projects
None yet
3 participants

The section on databases (https://github.com/nodejitsu/handbook/#databases) gives the redis client code as

var client = redis.createClient('subdomain.redistogo.com, 5309);

but according to node_redis docu (https://github.com/mranney/node_redis/#rediscreateclientport-host-options) the correct syntax should be redis.createClient(port, host, options), ie.

var client = redis.createClient(5309, 'subdomain.redistogo.com');

Likely a minor typo on the author's part, hope this helps other node_redis newbies.

Contributor

mmalecki commented Aug 11, 2012

@jesusabdullah Assigning to you

@ghost ghost assigned jfhbrook Aug 11, 2012

jfhbrook added a commit that referenced this issue Aug 11, 2012

Contributor

jfhbrook commented Aug 11, 2012

Should be fixed. This copy error may exist in jitsu as well, if that PR got merged.

I'll try to look later today.

@jfhbrook jfhbrook closed this Aug 11, 2012

Contributor

jfhbrook commented Aug 11, 2012

(Thanks @priscillahuang !)

no prob, glad to be of help! :)

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