Re: A bug with ALTER TABLE SET WITHOUT OIDS in CVS HEAD - Mailing list pgsql-hackers

From Tom Lane
Subject Re: A bug with ALTER TABLE SET WITHOUT OIDS in CVS HEAD
Date
Msg-id 5473.1227799385@sss.pgh.pa.us
Whole thread Raw
In response to Re: A bug with ALTER TABLE SET WITHOUT OIDS in CVS HEAD  (KaiGai Kohei <kaigai@ak.jp.nec.com>)
Responses Re: A bug with ALTER TABLE SET WITHOUT OIDS in CVS HEAD  (KaiGai Kohei <kaigai@ak.jp.nec.com>)
List pgsql-hackers
KaiGai Kohei <kaigai@ak.jp.nec.com> writes:
> If my understanding is correct, the following patch can fix the matters.

> !     if (ExecContextForcesOids(ps, &hasoid) &&
> !         hasoid != tupdesc->tdhasoid)
>           return false;
> --- 243,249 ----
> !     if (ExecContextForcesOids(ps, &hasoid))
>           return false;

This isn't fixing anything, it's just making the executor stick its
head in the sand.
        regards, tom lane


pgsql-hackers by date:

Previous
From: "Jaime Casanova"
Date:
Subject: Re: Fwd: [PATCHES] Auto Partitioning Patch - WIP version 1
Next
From: Emmanuel Cecchet
Date:
Subject: Re: Fwd: [PATCHES] Auto Partitioning Patch - WIP version 1