Tom Lane <t...@sss.pgh.pa.us> wrote: > Kevin Grittner <kgri...@ymail.com> writes: >> I failed to touch everything necessary to prevent MVs from >> having OIDs. This patch fixes the reported problem, and doesn't >> leave any gaps as far as I know; but I will do additional review >> to try to catch any other omissions. I figured I should address >> the reported problem now, though. > >> Will push later today if there are no objections. > > I object --- that's not a fix, that's a crude hack. It should > not be necessary to introduce relkind tests there. Determination > of whether OIDs exist in the target table should happen well > upstream, ie in whatever is constructing the intoClause. > Otherwise we'll be fixing code that examines the intoClause until > doomsday.
OK. I started doing it that way, but saw how much more code was changed than this way and gave in to an impulse to do a minimal change. I really need to resist that impulse more.... -- Kevin Grittner EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers