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
>
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 |