#33379: Add minimum database version checks
-------------------------------------+-------------------------------------
     Reporter:  Tim Graham           |                    Owner:  Hasan
                                     |  Ramezani
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  dev
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * needs_better_patch:  0 => 1


Comment:

 As noted on the PR, I tested this with CockroachDB and it doesn't work
 because the system check framework runs after other initialization
 queries, some of which fail on older versions of CockroachDB. Perhaps
 moving the error to another method like
 DatabaseWrapper.init_connection_state() as I did for CockroachDB
 (implementation linked in ticket description) is more suitable.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/33379#comment:9>
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.9aff0dc1947b519113c6944f1f9f78d3%40djangoproject.com.

Reply via email to