diff options
| author | Simon Charette <charette.s@gmail.com> | 2020-05-14 00:14:48 -0400 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2020-05-14 09:31:15 +0200 |
| commit | adfbf653dc1c1d0e0dacc4ed46602d22ba28b004 (patch) | |
| tree | dad32d2c85cf7d0df06eb0a1551570ad07f2cf8b /django/db/models/sql/query.py | |
| parent | 42c08ee46539ef44f8658ebb1cbefb408e0d03fe (diff) | |
Fixed #31568 -- Fixed alias reference when aggregating over multiple subqueries.
691def10a0197d83d2d108bd9043b0916d0f09b4 made all Subquery() instances
equal to each other which broke aggregation subquery pushdown which
relied on object equality to determine which alias it should select.
Subquery.__eq__() will be fixed in an another commit but
Query.rewrite_cols() should haved used object identity from the start.
Refs #30727, #30188.
Thanks Makina Corpus for the report.
Diffstat (limited to 'django/db/models/sql/query.py')
| -rw-r--r-- | django/db/models/sql/query.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py index f8e146b8de..375e22c4de 100644 --- a/django/db/models/sql/query.py +++ b/django/db/models/sql/query.py @@ -390,7 +390,7 @@ class Query(BaseExpression): else: # Reuse aliases of expressions already selected in subquery. for col_alias, selected_annotation in self.annotation_select.items(): - if selected_annotation == expr: + if selected_annotation is expr: new_expr = Ref(col_alias, expr) break else: |