|From:||asjo(at)koldfront(dot)dk (Adam =?iso-8859-1?Q?Sj=F8gren?=)|
|Subject:||Re: plperl - put schema-name in $_TD|
|Views:||Raw Message | Whole Thread | Download mbox|
On Wed, 24 May 2006 10:12:17 -0400, Andrew wrote:
> Patches should be made against the HEAD branch in CVS, not against a
> distro source.
Ok; I'll do that. (The patch did apply cleanly to CVS, though, but
On Wed, 24 May 2006 15:41:07 -0400, Tom wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>> This seems like a good idea, but we should probably make analogous
>> changes for plpgsql, pltcl and plpython. Having different trigger data
>> available in some of these doesn't seem like a good idea.
> Yeah. I'm also a little disturbed by using "nspname" which is an
> entirely internal name; plus it's a bit unclear *which* schema it's
> supposed to be. (One might think it's the schema the trigger function
> is in, for instance.) Somebody established a bad precedent by using
> "relname" for the table name.
I wasn't sure what to call it, so I modelled my change after relname ~
SPI_getrelname and arrived at the questionable nspname ~ SPI_getnspname.
> Maybe we should use field names like "table_name" and "table_schema".
> "relname" could be kept around for awhile but deprecated as a duplicate
> of "table_name".
On Thu, 25 May 2006 16:06:12 -0000, Greg wrote:
> +1 for table_schema and table_name, especially if in the future we provide
> things like trigger_schema.
I've attached a new patch, against CVS, that adds table_name and
table_schema instead, and updates the doc accordingly.
I haven't looked at the other languages as I do not use them; let me
know if I should take a stab at providing patches for them as well.
Thanks for your comments.
"Our hero regains consciousness at the feet of a Adam Sjøgren
sarcastic alien..." asjo(at)koldfront(dot)dk
|Next Message||Alvaro Herrera||2006-05-25 23:14:26||Re: [WIP] The relminxid addition, try 3|
|Previous Message||Chris Browne||2006-05-25 22:39:15||plpgsql documentation|