Merge pull request #140 from kawanet/single-server

avoid calculating hashCode for single server
memcachier · Dec 17, 2020 · 368a57e · 368a57e
2 parents 3155b9a + 6f5bd14
commit 368a57e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/memjs/memjs.js b/lib/memjs/memjs.js
@@ -86,12 +86,13 @@ Client.create = function(serversStr, options) {
 Client.prototype.server = function(key) {
   // TODO(alevy): should use consistent hashing and/or allow swapping hashing
   // mechanisms
-  var origIdx = hashCode(key) % this.servers.length;
+  var total = this.servers.length;
+  var origIdx = (total > 1) ? (hashCode(key) % total) : 0;
   var idx = origIdx;
   var serv = this.servers[idx];
   while (serv.wakeupAt &&
       serv.wakeupAt > Date.now()) {
-    idx = (idx + 1) % this.servers.length;
+    idx = (idx + 1) % total;
     if (idx === origIdx) {
       return null;
     }