[Django] #34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.

12 views
Skip to first unread message

Django

unread,
Nov 22, 2022, 6:43:27 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz | Owner: Mariusz Felisiak
Felisiak |
Type: Bug | Status: assigned
Component: Database | Version: 4.1
layer (models, ORM) |
Severity: Release | Keywords:
blocker |
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
`QuerySet.bulk_create()` crashes on `"pk"` in `unique_fields` which should
be allowed.

Bug in 0f6946495a8ec955b471ca1baaf408ceb53d4796.

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

Django

unread,
Nov 22, 2022, 7:08:37 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz

| Felisiak
Type: Bug | Status: assigned
Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Release blocker | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0

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

* has_patch: 0 => 1


Comment:

[https://github.com/django/django/pull/16317 PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/34177#comment:1>

Django

unread,
Nov 22, 2022, 7:32:02 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz
| Felisiak
Type: Bug | Status: assigned
Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Release blocker | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Description changed by Mariusz Felisiak:

Old description:

> `QuerySet.bulk_create()` crashes on `"pk"` in `unique_fields` which
> should be allowed.
>
> Bug in 0f6946495a8ec955b471ca1baaf408ceb53d4796.

New description:

`QuerySet.bulk_create()` crashes on `"pk"` in `unique_fields` which should
be allowed.

{{{
File "/django/django/db/backends/utils.py", line 89, in _execute
return self.cursor.execute(sql, params)
django.db.utils.ProgrammingError: column "pk" does not exist
LINE 1: ...S (3127, 3, 3, 'c'), (3128, 4, 4, 'd') ON CONFLICT("pk") DO ...
}}}

Bug in 0f6946495a8ec955b471ca1baaf408ceb53d4796.

--

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

Django

unread,
Nov 22, 2022, 8:21:19 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz
| Felisiak
Type: Bug | Status: assigned
Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Release blocker | 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 Carlton Gibson):

* stage: Unreviewed => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/34177#comment:3>

Django

unread,
Nov 22, 2022, 8:27:10 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz
| Felisiak
Type: Bug | Status: closed

Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Release blocker | 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
-------------------------------------+-------------------------------------
Changes (by GitHub <noreply@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"7d5329852f19c6ae78c6f6f3d3e41835377bf295" 7d53298]:
{{{
#!CommitTicketReference repository=""
revision="7d5329852f19c6ae78c6f6f3d3e41835377bf295"
Fixed #34177 -- Fixed QuerySet.bulk_create() crash on "pk" in
unique_fields.

Bug in 0f6946495a8ec955b471ca1baaf408ceb53d4796.
}}}

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

Django

unread,
Nov 22, 2022, 8:27:39 AM11/22/22
to django-...@googlegroups.com
#34177: QuerySet.bulk_create() crashes on "pk" in unique_fields.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz
| Felisiak
Type: Bug | Status: closed
Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Release blocker | 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:"3b0a8ea2993a2b011ad57304b3f978947adb5981" 3b0a8ea]:
{{{
#!CommitTicketReference repository=""
revision="3b0a8ea2993a2b011ad57304b3f978947adb5981"
[4.1.x] Fixed #34177 -- Fixed QuerySet.bulk_create() crash on "pk" in
unique_fields.

Bug in 0f6946495a8ec955b471ca1baaf408ceb53d4796.
Backport of 7d5329852f19c6ae78c6f6f3d3e41835377bf295 from main
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/34177#comment:5>

Reply all
Reply to author
Forward
0 new messages