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!
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 |
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 |
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 |