Opened 5 years ago
Last modified 5 years ago
#32478 closed Bug
Double OuterRef in Subquery with Case broken in 3.X — at Initial Version
| Reported by: | Igor Pejic | Owned by: | nobody |
|---|---|---|---|
| Component: | Database layer (models, ORM) | Version: | 3.0 |
| Severity: | Normal | Keywords: | outerref, subquery |
| Cc: | Igor Pejic | Triage Stage: | Ready for checkin |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
After trying to migrate from 2.2. to 3.0, we experience a problem with the translation of queries in PostgreSQL which used to work in 2.2.
A breaking test is shown in: https://github.com/igorpejic/django/pull/1/files
From investigation this is related to: https://code.djangoproject.com/ticket/31094
and it seems the regression was introduced in: https://github.com/django/django/commit/fb3f034f1c63160c0ff13c609acd01c18be12f80
It seems that the combination of the "double OuterRef" with the "Case" logic is causing the problem.
Query looks like:
books_with_same_name_as_country = Book.objects.filter(
id__in=Subquery(
Book.objects.filter(
name=OuterRef(OuterRef('country__name')),
).values('id')
)
).values('id')[:1]
books_breakdown = Publisher.objects.annotate(total_books=Case(
When(
num_awards__gte=2,
then=Subquery(books_with_same_name_as_country, IntegerField())
),
When(
num_awards__lt=0,
then=Count('country__publishers')
),
))
Stack trace:
======================================================================
ERROR: test_group_by_subquery_annotation_with_conditional (aggregation.tests.AggregateTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/igor/django-repo/django/db/backends/utils.py", line 86, in _execute
return self.cursor.execute(sql, params)
psycopg2.errors.GroupingError: subquery uses ungrouped column "aggregation_country.name" from outer query
LINE 1: ..."id" FROM "aggregation_book" U0 WHERE U0."name" = "aggregati...
^
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/igor/django-repo/django/test/testcases.py", line 1229, in skip_wrapper
return test_func(*args, **kwargs)
File "/home/igor/django-repo/tests/aggregation/tests.py", line 1299, in test_group_by_subquery_annotation_with_conditional
self.assertEqual(books_breakdown.get(id=self.p1.id).total_books, 1)
File "/home/igor/django-repo/django/db/models/query.py", line 411, in get
num = len(clone)
File "/home/igor/django-repo/django/db/models/query.py", line 258, in __len__
self._fetch_all()
File "/home/igor/django-repo/django/db/models/query.py", line 1261, in _fetch_all
self._result_cache = list(self._iterable_class(self))
File "/home/igor/django-repo/django/db/models/query.py", line 57, in __iter__
results = compiler.execute_sql(chunked_fetch=self.chunked_fetch, chunk_size=self.chunk_size)
File "/home/igor/django-repo/django/db/models/sql/compiler.py", line 1154, in execute_sql
cursor.execute(sql, params)
File "/home/igor/django-repo/django/db/backends/utils.py", line 68, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "/home/igor/django-repo/django/db/backends/utils.py", line 77, in _execute_with_wrappers
return executor(sql, params, many, context)
File "/home/igor/django-repo/django/db/backends/utils.py", line 86, in _execute
return self.cursor.execute(sql, params)
File "/home/igor/django-repo/django/db/utils.py", line 90, in __exit__
raise dj_exc_value.with_traceback(traceback) from exc_value
File "/home/igor/django-repo/django/db/backends/utils.py", line 86, in _execute
return self.cursor.execute(sql, params)
django.db.utils.ProgrammingError: subquery uses ungrouped column "aggregation_country.name" from outer query
LINE 1: ..."id" FROM "aggregation_book" U0 WHERE U0."name" = "aggregati...
^