Changes between Version 1 and Version 2 of Ticket #34597, comment 7


Ignore:
Timestamp:
May 29, 2023, 8:21:58 AM (12 months ago)
Author:
Simon Charette

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34597, comment 7

    v1 v2  
    11Thanks for the investigation David!
    22
    3 The `AND "blog"."id" = "blog"."id"` is definitely something we want to elide and was mentioned in comment:1 as a bug but the reporter said that they were also getting poor results with the provided ORM examples that explicitly don't have this criteria (e.g. `~Exists(Blog.objects.filter(translation=None, id=OuterRef("id"))`) so I suggest we wait to see how well the SQL examples of comment:3 (that don't have this criteria) perform for the reporter.
     3The `AND "blog"."id" = "blog"."id"` is definitely something we want to elide and was mentioned in comment:1 as a bug but the reporter said that they were also getting poor results with the provided ORM examples that explicitly don't have this criteria (e.g. `~Exists(Blog.objects.filter(translation=None, id=OuterRef("id"))`) so I suggest we wait to see how well the SQL examples of comment:3 (that also don't have this criteria) perform for the reporter.
    44
    55In other words, the `AND "blog"."id" = "blog"."id"` part is definitely a bug we want to solve but if we are going to address this issue it'd be great to confirm it's the only factor to blame.
Back to Top