Re: ToDo: plpgsql plugin for query and expression verification - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: ToDo: plpgsql plugin for query and expression verification
Date
Msg-id 162867791002160809r74f2dcc3n4992a36adcd98ee9@mail.gmail.com
Whole thread Raw
In response to Re: ToDo: plpgsql plugin for query and expression verification  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
2010/2/16 Tom Lane <tgl@sss.pgh.pa.us>:
> Pavel Stehule <pavel.stehule@gmail.com> writes:
>> I don't would to execute function - it is useless because you need
>> good UI for execution all path. My idea is different. gram.y has
>> check_sql_expr rutine. This is used for parser checking every static
>> SQL fragment in plpgsql function. With some hook we can do full plan
>> generation instead.
>
> Previous proposals in this line have foundered on examples like
> functions that create a temp table and then manipulate it.
> Only DDL-free functions can be statically checked in the way
> you suggest.
>

No and yes.

yes - 100% test are possible only on a) DDL free functions, b) 100%
static schema.
no - in reality schema is usually stable and we are able to check sql
using stable schema.

This proposal isn't about ideal checking - it isn't possible. It is
about the maximum from what is possible.

I would to identify bugs in not often using execution path before
production. This case is real. Stored procedures works well and after
half of year we finding broken identifiers in some queries.

> Between that and the parameter-related limitations that Hitoshi
> points out, the use case seems to be rather restricted ...

why? why is it better? do you have a way for runtime checking of all
possible execution path?

regards
Pavel

>
>                        regards, tom lane
>


pgsql-hackers by date:

Previous
From: "Greg Sabino Mullane"
Date:
Subject: Re: Listen / Notify - what to do when the queue is full
Next
From: Bruce Momjian
Date:
Subject: Re: CommitFest Status Summary - 2010-02-14