Opened 11 years ago

Last modified 5 years ago

#23027 new New feature

Add database-backend-specific checks

Reported by: Shai Berger Owned by: nobody
Component: Core (System checks) Version: dev
Severity: Normal Keywords:
Cc: Michael Manfre, Ahmad Abdallah Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

Different database backends have different features and behaviors. As a result, some definitions which may be ok on some backends, may warrant warnings or even errors on other backends.

As one example, a 65-chars-long field name is a problem on PostgreSQL, but not on any other core backend; as another, column named "timestamp" warrants a warning on Oracle (it might work or not...) but not anywhere else.

The place where such definitions are checked is the checks framework; and to account for the different behaviors, we need database-backend-specific checks.

This is a spin-off from #13711 -- which, in its last iteration, needed special care for the field-name-too-long check.

According to the ticket's flags, the next step(s) to move this issue forward are:

  • To provide a patch by sending a pull request. Claim the ticket when you start working so that someone else doesn't duplicate effort. Before sending a pull request, review your work against the patch review checklist. Check the "Has patch" flag on the ticket after sending a pull request and include a link to the pull request in the ticket comment when making that update. The usual format is: [https://github.com/django/django/pull/#### PR].

Change History (4)

comment:1 by Tim Graham, 11 years ago

Triage Stage: UnreviewedAccepted

comment:2 by Tim Graham, 11 years ago

I haven't looked into details, but I just came across django/db/backends/mysql/validation.py which implements MySQL specific field validation.

comment:3 by Michael Manfre, 10 years ago

Cc: Michael Manfre added

comment:4 by Ahmad Abdallah, 5 years ago

Cc: Ahmad Abdallah added
Note: See TracTickets for help on using tickets.
Back to Top