Merge pull request #61 from sliit-foss/master

Added match stage before facet to improve performance
aravindnc · Jul 5, 2024 · dde53c3 · dde53c3
2 parents 419db51 + d65250f
commit dde53c3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/mongoose-aggregate-paginate.js b/lib/mongoose-aggregate-paginate.js
@@ -89,7 +89,7 @@ function aggregatePaginate(query, options, callback) {
  const allowDiskUse = options.allowDiskUse || false;
  const isPaginationEnabled = options.pagination === false ? false : true;
 
- const q = this.aggregate();
+ let q = this.aggregate();
 
  if (query.options) q.options = query.options;
 
@@ -125,7 +125,13 @@ function aggregatePaginate(query, options, callback) {
 
  let promise;
  if (options.useFacet && !options.countQuery) {
- const [pipeline, countPipeline] = constructPipelines();
+ let [pipeline, countPipeline] = constructPipelines();
+ const match = pipeline[0]?.$match;
+ if (match) {
+ pipeline.shift();
+ countPipeline.shift();
+ q = q.match(match);
+ }
  promise = q
  .facet({
  docs: pipeline,

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "mongoose-aggregate-paginate-v2",
- "version": "1.1.0",
+ "version": "1.1.1",
  "description": "A cursor based custom aggregate pagination library for Mongoose with customizable labels.",
  "main": "index.js",
  "types": "types/index.d.ts",