#36118: Some usage of bulk_batch_size might not account for composite primary 
keys
on SQLite
-------------------------------------+-------------------------------------
     Reporter:  Simon Charette       |                     Type:  Bug
       Status:  new                  |                Component:  Database
                                     |  layer (models, ORM)
      Version:  5.2                  |                 Severity:  Release
                                     |  blocker
     Keywords:                       |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
 Brought up by Jacob Tyler Walls when reviewing proposed changes for
 #36116.

 -----

 While the initially surfaced instance related to a distinct issue (see
 ticket:27833#comment:13) there are definitive problems at least with
 `bulk_update` which
 
[https://github.com/django/django/blob/22fc151bb86a553d84c62d7effd289356e9b6c6c/django/db/models/query.py#L895
 doesn't expand] `pk` into multiple fields before calling `bulk_batch_size`
 and could result in `OperationalError` due to too many parameters on
 SQLite.

 The `bulk_update` case should be solved and other usage of
 `bulk_batch_size` audited.
-- 
Ticket URL: <https://code.djangoproject.com/ticket/36118>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion visit 
https://groups.google.com/d/msgid/django-updates/010701948506aa9b-3c569a44-361e-4e03-b6d9-63a1949b4251-000000%40eu-central-1.amazonses.com.

Reply via email to