summaryrefslogtreecommitdiff
path: root/django/db/models
diff options
context:
space:
mode:
authorClaude Paroz <claude@2xlibre.net>2013-03-20 10:32:13 +0100
committerClaude Paroz <claude@2xlibre.net>2013-03-20 10:43:14 +0100
commitb91067d9aa42e31d4375e00a703beaacdb30d608 (patch)
treead59a54a9d612225efd7818dff69e2d0ee0f1ff0 /django/db/models
parent5c788770167bc236fb1e16ac6d1414db26fee26c (diff)
[1.5.x] Revert "Fixed #19895 -- Made second iteration over invalid queryset raise an exception too"
This reverts commit d1e87eb3baf75b1b6a0ada46a9b77f7e347cdb60. This commit was the cause of a memory leak. See ticket for more details. Thanks Anssi Kääriäinen for identifying the source of the bug.
Diffstat (limited to 'django/db/models')
-rw-r--r--django/db/models/query.py14
1 files changed, 1 insertions, 13 deletions
diff --git a/django/db/models/query.py b/django/db/models/query.py
index b46528599e..952739e939 100644
--- a/django/db/models/query.py
+++ b/django/db/models/query.py
@@ -102,7 +102,7 @@ class QuerySet(object):
len(self)
if self._result_cache is None:
- self._iter = self._safe_iterator(self.iterator())
+ self._iter = self.iterator()
self._result_cache = []
if self._iter:
return self._result_iter()
@@ -337,18 +337,6 @@ class QuerySet(object):
yield obj
- def _safe_iterator(self, iterator):
- # ensure result cache is cleared when iterating over a queryset
- # raises an exception
- try:
- for item in iterator:
- yield item
- except StopIteration:
- raise
- except Exception:
- self._result_cache = None
- raise
-
def aggregate(self, *args, **kwargs):
"""
Returns a dictionary containing the calculations (aggregation)