Re: review: CHECK FUNCTION statement

Hello

there is fixed version

Regards

Pavel

2011/12/9 Albe Laurenz <laurenz(dot)albe(at)wien(dot)gv(dot)at>:
> Pavel Stehule wrote:
>> updated version
>>
>> changes:
>>
>> * CHECK FUNCTION ALL; is enabled - in this case functions from
>> pg_catalog schema are ignored
>>
>> I looked on parser, and I didn't other changes there - IN SCHEMA, FOR
>> ROLE are used more time there, so our usage will be consistent
>
>> a small addition
>>
>> * don't check SQL functions - are checked well now
>> * don't check functions from information_schema too
>
> One hunk in the patch fails due to conflict with
> commit d5f23af6bfbc454e86dd16e5c7a0bfc0cf6189d0
> (Peter Eisentraut's const patch).
>
> There are also compiler warnings about discarded const
> qualifiers in backend/nodes/copyfuncs.c,
> backend/nodes/equalfuncs.c and backend/parser/gram.y.
>
> There is a bug when ALL IN SCHEMA or ALL IN LANGUAGE
> is used:
>
> test=> CHECK FUNCTION ALL IN LANGUAGE plpgsql;
> ERROR:  language "language" does not exist
> test=> CHECK FUNCTION ALL IN SCHEMA laurenz;
> ERROR:  schema "schema" does not exist
>
> Something gets mixed up here.
>
> I like the idea that CHECK FUNCTION ALL without additional
> clauses works and ignores pg_catalog and information_schema!
>
> I'm working on some documentation, but it won't be final before
> the functionality is agreed upon.
>
> Yours,
> Laurenz Albe