Switch out ConcurrencyManager for Manager.
Review Request #14073 — Created Aug. 4, 2024 and submitted — Latest diff uploaded
Early on in Django history, the built-in
Manager.get_or_createmethod
was racy, potentially raisingIntegrityError. We had built a
ConcurrencyManagerto handle this case, and used it for many of our
models where we encountered issues.As of quite some time ago, Django's built-in
Managerdoes exactly the
same thing as ourConcurrencyManagerdid, and we just didn't discover
that because everything was working as expected. We've deprecated the
ConcurrencyManager as a result. This change swaps out all our usage to
just use the built-inManagerclass.
Ran unit tests.