#27441: MySQL backend output for 'DROP COLUMN' uses undocumented SQL
------------------------------------------------+------------------------
               Reporter:  Adam Chainz           |          Owner:  nobody
                   Type:  Cleanup/optimization  |         Status:  new
              Component:  Migrations            |        Version:  1.10
               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                     |
------------------------------------------------+------------------------
 Django's default SchemaEditor outputs `DROP COLUMN ... CASCADE`. When
 inspecting the SQL for a migration I saw this and wondered what the
 `CASCADE` means - it's not documented on MySQL (
 http://dev.mysql.com/doc/refman/5.7/en/alter-table.html ), but I
 discovered it is a no-op added for compatibility with e.g. postgres where
 it does mean something. I added this to the MariaDB documentation (
 https://mariadb.com/kb/en/mariadb/alter-table/ ) because it's editable,
 but it's not going to get added to the MySQL documentation any time soon.

 Since it's potentially confusing and a no-op, I think Django shouldn't
 output `CASCADE` on MySQL.

--
Ticket URL: <https://code.djangoproject.com/ticket/27441>
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/053.515c3db35aa9533a16690cdf9b7847d6%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to