#27690: remove sleep before dropping test db?
---------------------------------------------+------------------------
               Reporter:  David Szotten      |          Owner:  nobody
                   Type:  Uncategorized      |         Status:  new
              Component:  Testing framework  |        Version:  master
               Severity:  Normal             |       Keywords:
           Triage Stage:  Unreviewed         |      Has patch:  0
    Needs documentation:  0                  |    Needs tests:  0
Patch needs improvement:  0                  |  Easy pickings:  0
                  UI/UX:  0                  |
---------------------------------------------+------------------------
 Whilst looking at why a test run (with a small number of tests) was being
 slow, i discovered
 
https://github.com/django/django/blob/master/django/db/backends/base/creation.py#L281

 which calls `sleep(1)` before dropping the (test) db " to avoid "database
 is being accessed by other users" errors.".

 it looks like this line has been there since the beginning of the test
 framework:
 
https://github.com/django/django/commit/7dce86ce0220ffb9f3f579cbd1e881e988764c9d

 is this still needed? the few things i've tried seem to work fine without
 the sleep, but i'm not sure what other tests/checks might help make sure

--
Ticket URL: <https://code.djangoproject.com/ticket/27690>
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 post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/055.25a7004631d2d0171f696307327c100d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to