Opened 8 years ago

Closed 7 years ago

Last modified 5 years ago

#9919 closed (fixed)

Raw SQL documentation doesn't mention transaction management

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

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 8 years ago.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Leo Shklovskii

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

comment:1 Changed 8 years ago by Shai Berger

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

comment:2 Changed 8 years ago by Jacob

Resolution: duplicate
Status: newclosed

Duplicate of #9964.

comment:3 Changed 7 years ago by Russell Keith-Magee

milestone: 1.1
Patch needs improvement: set
Resolution: duplicate
Status: closedreopened
Triage Stage: UnreviewedAccepted

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 Changed 7 years ago by Russell Keith-Magee

Resolution: fixed
Status: reopenedclosed

(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 Changed 7 years ago by Russell Keith-Magee

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

comment:6 Changed 5 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

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