Re: Function with defval returns error

From: "Rushabh Lathia" <rushabh(dot)lathia(at)gmail(dot)com>
To: "Peter Eisentraut" <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, rushabh(dot)lathia(at)enterprisedb(dot)com
Subject: Re: Function with defval returns error
Date: 2008-12-16 06:32:24
Message-ID: 460abcb10812152232lefa0e18t78a8236c051eb2ec@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Another issue found on CVS head ....

CREATE USER test WITH PASSWORD 'test';
CREATE SCHEMA AUTHORIZATION test;

CREATE OR REPLACE FUNCTION f_test(x in numeric) RETURNS numeric as $$
BEGIN
RETURN x;
END;
$$ language plpgsql;

select f_test(10);

\c postgres test;

select f_test(10);

CREATE OR REPLACE FUNCTION f_test(x in numeric, y in varchar default 'Local
Function with parameters') RETURNs numeric as $$
BEGIN
RETURN x+1;
END;
$$ language plpgsql;

postgres=> select f_test(10);
ERROR: cache lookup failed for type 2139062142

On Tue, Dec 16, 2008 at 2:07 AM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:

> On Monday 15 December 2008 15:43:00 Tom Lane wrote:
> > Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> > > Rushabh Lathia wrote:
> > >> I think this should not return error as the input args here is
> > >> timestamp... inputs?
> > >
> > > In theory yes, but it's currently not that smart.
> >
> > This is truly horrid. Was that patch *really* ready to commit?
> > I noticed some comments added to polymorphism.sql that certainly
> > look like there's still a lot of half-bakedness in it.
>
> There is that one case where a call that could be allowed is
> overly-cautiously
> rejected. That only happens if you have a mix of overloading and default
> parameters. It's not really half-baked in the sense that it is not
> digestible; it's just not the greatest cake yet. It's
> improvement-compatible.
>

--
Rushabh Lathia

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nikhil Sontakke 2008-12-16 06:35:11 Re: Fwd: [PATCHES] Auto Partitioning Patch - WIP version 1
Previous Message sai srujan 2008-12-16 06:01:36 How to insert the values of .sql file (dump file) into postgresql 8.1