Changes between Initial Version and Version 1 of Ticket #34597, comment 7
- Timestamp:
- May 29, 2023, 8:21:41 AM (18 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #34597, comment 7
initial v1 1 1 Thanks for the investigation David! 2 2 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 (that don't have this criteria) perform for the reporter.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. 4 4 5 5 In 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.