Horton

Re: Materialized view rewrite is broken when there is an event trigger

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Önder Kalacı <onderkalaci@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-08-09T13:13:21Z
Lists: pgsql-hackers 
On Tue, Aug 09, 2022 at 02:55:06PM +0200, Önder Kalacı wrote:
> Attached a patch to fix as well. If the patch looks good to you, can you
> consider getting this to PG 15?

Thanks, this one is on me so I have added an open item.  I will
unfortunately not be able to address that this week because of life,
but I should be able to grab a little bit of time next week to look at
what you have.

Please note that we should not add an event in create_am.sql even if
it is empty, as it gets run in parallel of other tests so there could
be interferences.  I think that this had better live in
sql/event_trigger.sql, even if it requires an extra table AM to check
this specific case.
--
Michael

Commits

  1. Allow event trigger table_rewrite for ALTER MATERIALIZED VIEW