Re: [HACKERS] Patching dblink.c to avoid warning about open transaction

2005-09-28 Thread Jonathan Beit-Aharon
Joe Conway wrote: Bruce Momjian wrote: [ Joe, would you review this? ] Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. The pa

Re: [HACKERS] Patching dblink.c to avoid warning about open transaction

2005-09-26 Thread Joe Conway
Bruce Momjian wrote: [ Joe, would you review this? ] Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. The patch itself is pret

Re: [HACKERS] Patching dblink.c to avoid warning about open transaction

2005-09-26 Thread Bruce Momjian
[ Joe, would you review this? ] Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. ---

Re: [HACKERS] Patching dblink.c to avoid warning about open transaction

2005-09-23 Thread Bruce Momjian
Tom Lane wrote: > Jonathan Beit-Aharon <[EMAIL PROTECTED]> writes: > >     if (!conn) > >     > > DBLINK_CONN_NOT_AVAIL; > > Could we see this in a less broken format? Here is the patch in text format. -- Bruce Momjian| http://candle.pha.pa.us pgman

Re: [HACKERS] Patching dblink.c to avoid warning about open transaction

2005-09-23 Thread Tom Lane
Jonathan Beit-Aharon <[EMAIL PROTECTED]> writes: >     if (!conn) >     > DBLINK_CONN_NOT_AVAIL; > + > +   if (rcon) > +   > rcon->remoteTrFlag = (PQtransactionStatus(conn) != > PQTRANS_IDLE); > > -   res = PQexec(conn, "BEGIN"); > -   if (PQresultStatus(r