Re: [PL/pgSQL] function call

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Merlin Moncure <mmoncure(at)gmail(dot)com>
Cc: Tarlika Elisabeth Schmitz <postgresql6(at)numerixtechnology(dot)de>, pgsql-general <pgsql-general(at)postgresql(dot)org>
Subject: Re: [PL/pgSQL] function call
Date: 2011-10-31 15:01:53
Message-ID: CAFj8pRDw1E0qQA+5R1zvLu0T4t86CK-4=hrknZBW8oO2hQjTfA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general

2011/10/31 Merlin Moncure <mmoncure(at)gmail(dot)com>:
> On Mon, Oct 31, 2011 at 8:31 AM, Tarlika Elisabeth Schmitz
> <postgresql6(at)numerixtechnology(dot)de> wrote:
>> I have created a function log_insert(), which is simply a shorthand for
>> an INSERT table and which I want to call from various trigger functions.
>>
>> CREATE OR REPLACE FUNCTION log_insert(vseverity text, vtrigger text,
>> vtriggertable text, vtriggerid text, vmessage text) RETURNS boolean AS
>> $BODY$
>> BEGIN
>>  INSERT INTO log
>>  (severity, trigger,triggertable, triggerid, message)
>>  VALUES
>>  (vseverity, vtrigger,vtriggertable, vtriggerid, vmessage);
>> END
>> $BODY$
>> LANGUAGE plpgsql VOLATILE;
>>
>>
>> I tried:
>> log_insert('I', TG_NAME, TG_TABLE_NAME, NEW.id, 'some msg');
>> => I get a syntax error on CREATE TRIGGER.
>>
>> SELECT log_insert(...)
>> => passes the syntax check but throws an error when run:
>> "function log_insert(unknown, unknown, unknown, integer, unknown) does
>> not exist Hint: No function matches the given name and argument types.
>> You might need to add explicit type casts."
>>
>>
>> Any help would be greatly appreciated.
>
> There is some context you are not passing here -- the log_insert
> function is being inside a trigger function which is where your error
> always is.  However, in pl/pgsql, you always call functions with
> PERFORM or SELECT depending if you want to process the result.
>
> also, FWIW, I don't like a simple wrapper for insert statement like
> that -- the syntax brevity is outweighed by the loss of SQL features
> such as being able to pass DEFAULT for columns.
>

you can use a PL default parameters now. And when there are lot of
parameters a named notation is useful

regards

Pavel

> merlin
>
> --
> Sent via pgsql-general mailing list (pgsql-general(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-general
>

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Adrian Schreyer 2011-10-31 17:52:05 Query planner always has exactly half of the rows in the table as "plan rows" with new GIST operator class
Previous Message Merlin Moncure 2011-10-31 14:41:40 Re: [PL/pgSQL] function call