Re: [PATCHES] Small fix for _equalValue()

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Thomas Lockhart <thomas(at)fourpalms(dot)org>
Cc: Fernando Nasser <fnasser(at)redhat(dot)com>, PostgreSQL Hackers List <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] Small fix for _equalValue()
Date: 2002-03-09 23:58:56
Message-ID: 19332.1015718336@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Thomas Lockhart <thomas(at)fourpalms(dot)org> writes:
>> That's because I already committed the other changes he pointed out ;-).
>> But yeah, we seem to be copy-clean again.

> I had thought that you objected to the guard code in the copy functions
> since nodes should not have had the content they did. And afaik I have
> now fixed the upstream problems with the content.

Right, the SET DEFAULT problem is fixed that way. Fernando had pointed
out a couple of problems in unrelated constructs (GRANT and something
else I forget now) that also needed to be fixed. Those fixes did get
committed.

> Had you changed you mind about the necessity for the guard code?

No. I think Value is fine as-is.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Lockhart 2002-03-10 00:28:51 Re: [PATCHES] Small fix for _equalValue()
Previous Message Bruce Momjian 2002-03-09 23:49:20 Re: Rationalizing EXPLAIN VERBOSE output

Browse pgsql-patches by date

  From Date Subject
Next Message Thomas Lockhart 2002-03-10 00:28:51 Re: [PATCHES] Small fix for _equalValue()
Previous Message Thomas Lockhart 2002-03-09 23:43:16 Re: [PATCHES] Small fix for _equalValue()