Re: ERROR: argument to pg_get_expr() must come from system catalogs

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: ERROR: argument to pg_get_expr() must come from system catalogs
Date: 2010-07-16 13:49:28
Message-ID: 14739.1279288168@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> writes:
> On 13/07/10 21:36, Tom Lane wrote:
>> I wasn't terribly happy with that approach to begin with. I think we
>> need to rethink.

> Do you want to go ahead with your plan of changing what's passed in
> FuncInfo? I won't object if you want to do it, but I wouldn't feel
> comfortable with backporting such big changes myself.

I will take a look at it, but not right away. Since we have no
near-term plans for new minor releases, I don't think it's urgent.

> If we continue with the approach I took, we should implement the
> suggestion to create a new data type for this in 9.1. That would be more
> waterproof than the changes I made, if we introduce new ways to call
> functions in the future.

Agreed, that seems like a better solution going forward than either of
the others.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kevin Grittner 2010-07-16 13:55:19 Re: dividing money by money
Previous Message Kevin Grittner 2010-07-16 13:39:09 Re: dividing money by money