Re: [Django] #32143: Use EXISTS to exclude multi-valued relationships

Django

unread,

Oct 26, 2020, 1:20:13 AM10/26/20

to django-...@googlegroups.com

#32143: Use EXISTS to exclude multi-valued relationships
-------------------------------------+-------------------------------------
Reporter: Simon Charette | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Database layer | Version: 3.1
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* stage: Unreviewed => Accepted

--
Ticket URL: <https://code.djangoproject.com/ticket/32143#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,

Oct 27, 2020, 5:12:32 AM10/27/20

to django-...@googlegroups.com

#32143: Use EXISTS to exclude multi-valued relationships
-------------------------------------+-------------------------------------

Reporter: Simon Charette | Owner: Simon
Type: | Charette
Cleanup/optimization | Status: assigned

Component: Database layer | Version: 3.1
(models, ORM) |
Severity: Normal | Resolution:

Keywords: | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* owner: nobody => Simon Charette
* status: new => assigned
* stage: Accepted => Ready for checkin

--
Ticket URL: <https://code.djangoproject.com/ticket/32143#comment:2>

Django

unread,

Jun 14, 2023, 6:52:22 AM6/14/23

to django-...@googlegroups.com

#32143: Use EXISTS to exclude multi-valued relationships
-------------------------------------+-------------------------------------
Reporter: Simon Charette | Owner: Simon
Type: | Charette

Cleanup/optimization | Status: closed

Component: Database layer | Version: 3.1
(models, ORM) |

Severity: Normal | Resolution: fixed

Comment (by Mariusz Felisiak <felisiak.mariusz@…>):

In [changeset:"cfc9c94d97edf9aaea647bbecc89a791cf7ec0b8" cfc9c94]:
{{{
#!CommitTicketReference repository=""
revision="cfc9c94d97edf9aaea647bbecc89a791cf7ec0b8"
Refs #32143 -- Adjusted a comment about subquery usage in
Query.split_exclude().
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/32143#comment:4>

Django

unread,

Jun 14, 2023, 6:52:23 AM6/14/23

to django-...@googlegroups.com

#32143: Use EXISTS to exclude multi-valued relationships
-------------------------------------+-------------------------------------
Reporter: Simon Charette | Owner: Simon
Type: | Charette
Cleanup/optimization | Status: closed
Component: Database layer | Version: 3.1
(models, ORM) |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Mariusz Felisiak <felisiak.mariusz@…>):

In [changeset:"1c4f5f314e2b0c77b3fa0c75f703218e7e06f4be" 1c4f5f3]:
{{{
#!CommitTicketReference repository=""
revision="1c4f5f314e2b0c77b3fa0c75f703218e7e06f4be"
Refs #32143 -- Removed superflous constraints on excluded query.

The outer query reference is not necessary when alias can be reused and
can even be harmful by confusing query planers.