pub/mongoengine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

reinsert fix; add comments, reference

Browse Source
This commit is contained in:
Neeraj Suthar 2019-04-06 17:42:14 +05:30
parent 4ccfdf051d
commit 61081651e4
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mongoengine/queryset/base.py
View File

@ -1200,7 +1200,11 @@ class BaseQuerySet(object):
initial_pipeline.append({'$sort': dict(self._ordering)}) initial_pipeline.append({'$sort': dict(self._ordering)})
if self._limit is not None: if self._limit is not None:
initial_pipeline.append({'$limit': self._limit}) # As per MongoDB Documentation (https://docs.mongodb.com/manual/reference/operator/aggregation/limit/),
# keeping limit stage right after sort stage is more efficient. But this leads to wrong set of documents
# for a skip stage that might succeed these. So we need to maintain more documents in memory in such a
# case (https://stackoverflow.com/a/24161461).
initial_pipeline.append({'$limit': self._limit + (self._skip or 0)})
if self._skip is not None: if self._skip is not None:
initial_pipeline.append({'$skip': self._skip}) initial_pipeline.append({'$skip': self._skip})