Re: postgres_fdw: batch inserts vs. before row triggers

Поиск
Список
Период
Сортировка
От Matthias van de Meent
Тема Re: postgres_fdw: batch inserts vs. before row triggers
Дата
Msg-id CAEze2WhUbmfA8nbnCdLRu9d3FvYTqkgGgCwrLEw_rC+T1du16w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: postgres_fdw: batch inserts vs. before row triggers  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Wed, 3 Aug 2022 at 23:57, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Matthias van de Meent <boekewurm+postgres@gmail.com> writes:
> > I don't have a current version of the SQL spec, but one preliminary
> > version of SQL:2012 I retrieved via the wiki details that all BEFORE
> > triggers on INSERT/UPDATE/DELETE statements are all executed before
> > _any_ of that statements' affected data is modified.
> > ...
> > I don't know about the semantics of triggers in the latest SQL
> > standard versions, but based on that sample it seems like we're
> > non-compliant on BEFORE trigger behaviour, and it doesn't seem like
> > it's documented in the trigger documentation.
>
> I think we're compliant if you declare the trigger functions as
> stable (or immutable, but in any case where this matters, I think
> you'd be lying).  They'll then run with the snapshot of the calling
> query, in which those updates are not yet visible.
>
> This is documented somewhere, but maybe not anywhere near triggers.

Thank you for this pointer.

Looking around a bit, it seems like this behaviour for functions is
indeed documented in xfunc.sgml, but rendered docs page [0] does not
seem to mention triggers, nor does the triggers page link to that part
of the xfunc document. This makes it quite easy to overlook that this
is expected (?) behaviour for VOLATILE functions only.

Kind regards,

Matthias van de Meent

[0] https://www.postgresql.org/docs/current/xfunc-volatility.html



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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Unstable tests for recovery conflict handling
Следующее
От: Junwang Zhao
Дата:
Сообщение: Re: [doc] fix a potential grammer mistake