trivial patch: foreign table either defines row type

Lists: pgsql-hackers
From: Alexander Soudakov <cygakob(at)gmail(dot)com>
To: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: trivial patch: foreign table either defines row type
Date: 2011-11-12 15:54:42
Message-ID: CACCOprdGvK_P4o+G+YYdg7VCuq5Bb90cL2wzLbbj0Z5u6C2foQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hello Hackers.

I attached trivial patch.

Foreign table either defines row type. I simply added this type to
hardcoded check.
I faced this limitation while developing www fdw feature
(http://wiki.postgresql.org/wiki/WWW_FDW).

--
Alexander Soudakov
Software Developer
email: cygakoB(at)gmail(dot)com
skype: asudakov

Attachment Content-Type Size
foreign-table-as-argument-to-plpgsql.patch text/x-patch 874 bytes

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alexander Soudakov <cygakob(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: trivial patch: foreign table either defines row type
Date: 2011-11-12 23:50:48
Message-ID: 21730.1321141848@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Alexander Soudakov <cygakob(at)gmail(dot)com> writes:
> Foreign table either defines row type. I simply added this type to
> hardcoded check.

Yeah, I think this was just an oversight. (You missed a spot in
plpgsql_parse_cwordtype, though.) Patch applied.

regards, tom lane