Skip to content

Commit

Permalink
[minor] Add missing compatiblity options
Browse files Browse the repository at this point in the history
  • Loading branch information
3rd-Eden committed Sep 4, 2014
1 parent b075d4a commit c6c3ac9
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions lib/memcached.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,9 @@ function Client (args, options) {
Utils.merge(this, options);

this.servers = servers;
this.HashRing = new HashRing(args, this.algorithm);
this.HashRing = new HashRing(args, this.algorithm, {
compatiblity: this.compatiblity
});
this.connections = {};
this.issues = [];
}
Expand All @@ -71,7 +73,8 @@ Client.config = {
, maxValue: 1048576 // max length of value allowed by Memcached
, activeQueries: 0
, maxQueueSize: -1
, algorithm: 'md5' // hashing algorithm that is used for key mapping
, algorithm: 'md5' // hashing algorithm that is used for key mapping
, compatiblity: 'ketama' // hashring compatiblity

This comment has been minimized.

Copy link
@ronkorving

ronkorving Sep 8, 2014

Collaborator

It seems consistent, but it is a typo.

compatiblity -> compatibility


, poolSize: 10 // maximal parallel connections
, retries: 5 // Connection pool retries to pull connection from pool
Expand Down

1 comment on commit c6c3ac9

@3rd-Eden
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in master.

Please sign in to comment.