#27862 closed Bug (fixed)
Exists() feature generates invalid SQL query on postgres backend
Reported by: | Vasily Stepanov | Owned by: | Matthew Schinckel |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | 1.11 |
Severity: | Release blocker | Keywords: | Queryset Subquery Exists |
Cc: | Matthew Schinckel | 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 (last modified by )
Exists() feature, implemented in #27149 ticket, not working properly on postgres backend.
This happens only in complex queries, where django has to use table aliases.
Consider the following models:
class Foo(models.Model): spam1 = models.ForeignKey('Spam', on_delete=models.CASCADE, related_name='+') spam2 = models.ForeignKey('Spam', on_delete=models.CASCADE, related_name='+') class Bar(models.Model): name1 = models.CharField(max_length=200) name2 = models.CharField(max_length=200) class Spam(models.Model): name = models.CharField(max_length=200)
And the code:
bars = models.Bar.objects.filter( name1=OuterRef('spam1__name'), name2=OuterRef('spam2__name')) qs = models.Foo.objects.annotate(bars=Exists(bars)) qs = qs.filter(bars=True)
This SQL generated using django.db.backends.postgresql
backend:
SELECT "demo_foo"."id", "demo_foo"."spam1_id", "demo_foo"."spam2_id", EXISTS( SELECT U0."id", U0."name1", U0."name2" FROM "demo_bar" U0 WHERE (U0."name2" = ("demo_spam"."name") AND U0."name1" = ("T3"."name"))) AS "bars" FROM "demo_foo" INNER JOIN "demo_spam" ON ("demo_foo"."spam2_id" = "demo_spam"."id") INNER JOIN "demo_spam" T3 ON ("demo_foo"."spam1_id" = T3."id") WHERE EXISTS( SELECT U0."id", U0."name1", U0."name2" FROM "demo_bar" U0 WHERE (U0."name2" = ("demo_spam"."name") AND U0."name1" = ("T3"."name"))) = True
which ends up with this error:
django.db.utils.ProgrammingError: missing FROM-clause entry for table "T3" LINE 1: ...."name1" = ("demo_spam"."name") AND U0."name2" = ("T3"."name... ^
This happens because of quotes around T3
.
SQL works as expected, if you remove these quotes and execute it manually.
Change History (10)
comment:1 by , 8 years ago
Description: | modified (diff) |
---|
comment:2 by , 8 years ago
Severity: | Normal → Release blocker |
---|---|
Triage Stage: | Unreviewed → Accepted |
Version: | master → 1.11 |
comment:3 by , 8 years ago
Cc: | added |
---|
comment:4 by , 8 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
It looks to be a difference between the quoting of the alias in the tables clause. I'll have a look.
Nicely presented bug report!
comment:5 by , 8 years ago
Confirmed that it's a problem with Subquery, and not just Exists.
However, I don't believe Subquery is doing anything with the alias: I don't know where the quoting is being added.
comment:6 by , 8 years ago
Keywords: | Subquery added; SubQuery removed |
---|
Okay, progress. It seems that the alias is not in compiler.query.external_aliases
, but I think it should be.
comment:7 by , 8 years ago
Has patch: | set |
---|
So, I think I have a fix for this: it fixes this specific use case, but I'm at a loss as to if it will break something else.
comment:8 by , 8 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
Matt, are you interested in looking into this?