fix: make queryViewByUser more efficient

limit the potential number of requests made to couchdb by gradually increasing the limit used for each request Closes #30
cheminfo · Sep 8, 2016 · d228f44 · d228f44
1 parent 5555578
commit d228f44
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/index.js b/src/index.js
@@ -313,7 +313,7 @@ class Couch {
 
     async queryViewByUser(user, view, options, rights) {
         debug(`queryViewByUser (${user}, ${view})`);
-        options = options || {};
+        options = Object.assign({}, options);
         options.include_docs = true;
         options.reduce = false;
         options.skip = 0;
@@ -334,13 +334,13 @@ class Couch {
             if (!options.limit) return rows;
 
             // Concatenate
-            limit = options.limit;
-            options.skip += limit;
+            options.skip += options.limit;
+            options.limit = options.limit * 2;
             cumRows = cumRows.concat(rows);
         }
 
         // Get rid of extra rows
-        return cumRows.filter((r, idx) => idx < options.limit);
+        return cumRows.filter((r, idx) => idx < limit);
     }
 
     async getEntriesByUserAndRights(user, rights, options = {}) {