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

From: Joe Conway <mail(at)joeconway(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: David Fetter <david(at)fetter(dot)org>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Jonathan Beit-Aharon <jbeitaharon(at)intrusic(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] Patching dblink.c to avoid warning about
Date: 2005-10-07 14:38:21
Message-ID: 4346885D.8010303@joeconway.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
> David Fetter <david(at)fetter(dot)org> writes:
>
>>On Thu, Oct 06, 2005 at 10:38:54PM -0400, Bruce Momjian wrote:
>>
>>>I don't know if people want this for 8.1 or 8.2.
>
>>8.1, IMHO. It's a bug fix. :)
>
> Not unless Joe Conway signs off on it ...
>

I had asked on the original simple patch, and I'll ask again now -- why
is this needed? I don't have a clear understanding of the problem that
this (or the earlier) patch is trying to solve. Without that, it is
impossible to say whether it is a bug fix or feature.

Joe

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2005-10-07 14:52:31 Re: [HACKERS] Patching dblink.c to avoid warning about open
Previous Message Alvaro Herrera 2005-10-07 14:17:21 Re: [HACKERS] Shell script to extract a table from a plain text dump

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-10-07 14:52:31 Re: [HACKERS] Patching dblink.c to avoid warning about open
Previous Message David Fetter 2005-10-07 03:54:13 Re: [HACKERS] Patching dblink.c to avoid warning about open transaction