Your patch has been added to the PostgreSQL unapplied patches list at:
http://candle.pha.pa.us/cgi-bin/pgpatches I will try to apply it within the next 48 hours. --------------------------------------------------------------------------- David Van Wie wrote: > On Thu, 22 Aug 2002, Tom Lane wrote: > > > I believe our intention has been to modify the backend to offer that > > behavior (with a GUC variable to turn it on or off). Supporting it > > only in psql doesn't seem the way to go. > > After playing around with the transaction code for a while, I think I have > a reasonable idea of how to do this in the backend. This is my first time > working with the backend code, so it's entirely possible that I'm going > about this completely wrong, but the change seems to work and pass > regression tests. > > I added a GUC variable "autocommit" which defaults to true. (Current > behavior). When it is set to false, the backend should begin a transaction > block if it is not inside one when executing any command. I did this by > changing StartTransactionCommand() in backend/access/transam/xact.c to > call BeginTransactionBlock() after StartTransaction() if it is not > currently in a transaction block. > > Does this sound like a reasonable approach? > > My first attempt at a patch (to current cvs source) is attached. > > ...dave Content-Description: [ Attachment, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED] -- Bruce Momjian | http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073 ---------------------------(end of broadcast)--------------------------- TIP 4: Don't 'kill -9' the postmaster