#3214: [patch] raw sql file doesn't recognize quotes correctly
-------------------------------------+-------------------------------------
Reporter: shaunc <shaun@…> | Owner: nobody
Type: Bug | Status: new
Component: Database layer | Version: master
(models, ORM) | Resolution:
Severity: Release blocker | Triage Stage: Accepted
Keywords: rawsql migrations | Needs documentation: 0
Has patch: 0 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by aaugustin):
* keywords: rawsql => rawsql migrations
* needs_better_patch: 1 => 0
* has_patch: 1 => 0
* severity: Normal => Release blocker
Comment:
This ticket never tells which backend support multiple statements in
`cursor.execute()` and which don't. It would be interesting to check that
on the four officially supported databases before going any further...
----
If indeed some backends need splitting, in order to close this ticket, we
must implement a similar approach as #22401 in `RunSQL._split_sql`.
Splitting occurs only when explicitly setting `multiple=True`, which
Django itself never does, even in the test suite. We should add a test for
that (and skip it when sqlparse isn't available.
The patch will be easier if we include it in 1.7 because we won't need a
deprecation path. Therefore, marking as a release blocker.
--
Ticket URL: <https://code.djangoproject.com/ticket/3214#comment:41>
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 post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/086.11a34233b7e4fe3050e57f0d33b705c4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
