-
Notifications
You must be signed in to change notification settings - Fork 24
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
🧪 Add regression test for order race condition
This is extremely hard to actually reproduce... I'm sorry. Basically the idea is that multiple requests can arrive at the same time (and they will do because of how the frontend is coded!) which kicks off a number of threads that all start with the same view on the data. django-ordered-model then performs some UPDATE queries based on that view of the data and finally a save call to the object itself to set the order. That last action often corrects a lot of things, but the UPDATE query causes mayhem. See also django-ordered-model/django-ordered-model#184.
- Loading branch information
1 parent
b0e52ad
commit 870d023
Showing
1 changed file
with
197 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters