Re: [HACKERS] ECPG ignores SAVEPOINT if first statement of a transaction

From: Michael Meskes <meskes(at)postgresql(dot)org>
To: Michael Fuhr <mike(at)fuhr(dot)org>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, PostgreSQL-interfaces <pgsql-interfaces(at)postgresql(dot)org>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] ECPG ignores SAVEPOINT if first statement of a transaction
Date: 2005-09-12 12:36:34
Message-ID: 20050912123634.GB21008@1
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-interfaces pgsql-patches

On Sat, Aug 13, 2005 at 09:01:45AM -0600, Michael Fuhr wrote:
> If no statements have been executed in this transaction then committed
> is true, so ECPGtrans() ignores the current statement. It looks
> like the code should make an extra consideration for SAVEPOINT in
> case it's the first statement.

I rearranged the whole code as it seemed to have more holes. This code
really was old. :-)

So could you please check with the lates CVS, either HEAD or 8.0?

> I'm also wondering if the check for a NULL return value from PQexec()
> is sufficient. Shouldn't it also check for a non-NULL result that's
> anything other than PGRES_COMMAND_OK?

Yes, it should. I changed this as well.

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes(at)jabber(dot)org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Josh Berkus 2005-09-12 16:13:02 Re: Spinlocks, yet again: a new test case
Previous Message Tino Wildenhain 2005-09-12 10:03:33 Re: Materialized Views in PostgreSQL

Browse pgsql-interfaces by date

  From Date Subject
Next Message fabrizio picca 2005-09-12 14:17:33 help needed on SPI_modifytuple.
Previous Message Tom Lane 2005-09-11 00:09:46 Re: Change of default type in DBD::Pg 1.42

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2005-09-12 14:05:22 Re: SQL/XML publishing function experimental patch
Previous Message Pavel Stehule 2005-09-12 07:03:38 SQL/XML publishing function experimental patch