Opened 3 hours ago
Last modified 3 hours ago
#36118 new Bug
Some usage of bulk_batch_size might not account for composite primary keys on SQLite
| Reported by: | Simon Charette | Owned by: | |
|---|---|---|---|
| Component: | Database layer (models, ORM) | Version: | 5.2 |
| Severity: | Release blocker | Keywords: | |
| Cc: | Jacob Walls | Triage Stage: | Accepted |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
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 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.
Note:
See TracTickets
for help on using tickets.