Re: proposal: plpgsql - Assert statement

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: proposal: plpgsql - Assert statement
Дата
Msg-id CAFj8pRCx8vt7sR_kQkWJtSz9AyLp6s8dF6uw3BxbuzHF3M9Rzw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: proposal: plpgsql - Assert statement  (Petr Jelinek <petr@2ndquadrant.com>)
Ответы Re: proposal: plpgsql - Assert statement
Список pgsql-hackers
Hi

2014-10-14 22:57 GMT+02:00 Petr Jelinek <petr@2ndquadrant.com>:
On 09/09/14 17:37, Pavel Stehule wrote:
Ada is language with strong character, and PLpgSQL is little bit strange
fork - so it isn't easy to find some form, how to solve all requirements.

My requests:

* be consistent with current PLpgSQL syntax and logic
* allow some future extensibility
* allow a static analyses without hard expression processing

But I am thinking so there are some points where can be some agreement -
although it is not ASSERT implementation.

enhancing RAISE WHEN - please, see attached patch -

I prefer RAISE WHEN again RAISE WHERE due consistency with EXIT and
CONTINUE [ WHEN ];


Short review of the patch. Note that this has nothing to do with actual assertions, at the moment it's just enhancement of RAISE statement to make it optionally conditional. As I was one of the people who voted for it I do think we want this and I like the syntax.

Code applies cleanly, seems formatted according to project standards - there is unnecessary whitespace added in variable declaration part of exec_stmt_raise which should be removed.

fixed
 

Passes make check, I would prefer to have little more complex expression than just "true" in the new regression test added for this feature.

fixed 

Did some manual testing, seems to work as advertised.

There are no docs at the moment which is the only show-stopper that I can see so far.

fixed

Regards

Pavel
 

--
 Petr Jelinek                  http://www.2ndQuadrant.com/

 PostgreSQL Development, 24x7 Support, Training & Services

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Stephen Frost
Дата:
Сообщение: Re: CREATE POLICY and RETURNING
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: CREATE POLICY and RETURNING