Skip to content

Fixed #36430 -- Removed artificially low limit on single field bulk operations on SQLite. #19522

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Aug 21, 2025
Merged

Fixed #36430 -- Removed artificially low limit on single field bulk operations on SQLite. #19522

merged 2 commits into from
Aug 21, 2025
+27 −36

Conversation

jacobtylerwalls
Copy link
Member

Trac ticket number

ticket-36430

Branch description

Thanks @laymonage for analysis on the ticket about why this limit should no longer be relevant.

Checklist

  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number, and ends with a period.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

@jacobtylerwalls jacobtylerwalls force-pushed the sqlite-bulk-single-field branch 2 times, most recently from 208e3a9 to e1bb0bf Compare June 4, 2025 03:37
@jacobtylerwalls
Copy link
Member Author

buildbot, test on oracle.

@jacobtylerwalls jacobtylerwalls mentioned this pull request Jun 6, 2025
Merged
@jacobtylerwalls jacobtylerwalls force-pushed the sqlite-bulk-single-field branch from e1bb0bf to 4e0e25b Compare June 12, 2025 03:53
@sarahboyce sarahboyce force-pushed the sqlite-bulk-single-field branch 3 times, most recently from cba3892 to 26cbf1d Compare August 7, 2025 08:37
sarahboyce
sarahboyce reviewed Aug 7, 2025
View reviewed changes
@sarahboyce sarahboyce force-pushed the sqlite-bulk-single-field branch from 26cbf1d to 17dce75 Compare August 11, 2025 13:33
@sarahboyce
Copy link
Contributor

buildbot, test on oracle.

sarahboyce
sarahboyce approved these changes Aug 11, 2025
View reviewed changes
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, this looks good to me 👍

jacobtylerwalls
jacobtylerwalls commented Aug 11, 2025
View reviewed changes
@jacobtylerwalls jacobtylerwalls force-pushed the sqlite-bulk-single-field branch from 17dce75 to 56bdd31 Compare August 17, 2025 11:56
@jacobtylerwalls jacobtylerwalls force-pushed the sqlite-bulk-single-field branch from 56bdd31 to dd629fd Compare August 17, 2025 11:58
@jacobtylerwalls
Copy link
Member Author

buildbot, test on oracle.

sarahboyce
sarahboyce approved these changes Aug 19, 2025
View reviewed changes
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the updates ⭐

@sarahboyce sarahboyce merged commit d3cf24e into django:main Aug 21, 2025
34 checks passed
@jacobtylerwalls jacobtylerwalls deleted the sqlite-bulk-single-field branch August 21, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sarahboyce sarahboyce sarahboyce approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jacobtylerwalls @sarahboyce