Opened 16 years ago

Closed 16 years ago

Last modified 13 years ago

#9919 closed (fixed)

Raw SQL documentation doesn't mention transaction management

Reported by: Leo Shklovskii Owned by: nobody
Component: Documentation Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Attempting to do modifying raw SQL with a cursor requires the transaction to be explicitly committed. The documentation at: ​http://docs.djangoproject.com/en/dev/topics/db/sql/ doesn't mention anything about this.

I've attached a patch that documents this detail and cleaned up some of the style of that page to make it more legible.

Attachments (1)

raw-sql-docs-r9690.patch​ (4.2 KB ) - added by Leo Shklovskii 16 years ago.

Download all attachments as: .zip

Change History (7)

by Leo Shklovskii, 16 years ago

Attachment: raw-sql-docs-r9690.patch​ added

comment:1 by Shai Berger, 16 years ago

It's not the docs that are wrong; it's the code. See #9964.

comment:2 by Jacob, 16 years ago

Resolution: → duplicate
Status: new → closed

Duplicate of #9964.

comment:3 by Russell Keith-Magee, 16 years ago

milestone: → 1.1
Patch needs improvement: set
Resolution: duplicate
Status: closed → reopened
Triage Stage: Unreviewed → Accepted

In the absence of a resolution for #9964, we should add a note on the need to manually dirty transactions if you use raw-sql.

The current patch doesn't do that, though.

comment:4 by Russell Keith-Magee, 16 years ago

Resolution: → fixed
Status: reopened → closed

(In [11024]) [1.0.X] Fixed #9919 -- Added note on the need to mark transactions as dirty when using raw SQL.

Merge of r11022 from trunk.

comment:5 by Russell Keith-Magee, 16 years ago

(In [11022]) Fixed #9919 -- Added note on the need to mark transactions as dirty when using raw SQL.

comment:6 by Jacob, 13 years ago

milestone: 1.1

Milestone 1.1 deleted

Note: See TracTickets for help on using tickets.
Back to Top