Re: BUG #15044: materialized views incompatibility with logicalreplication in postgres 10

On 2/17/18 22:43, Peter Eisentraut wrote:
> On 2/5/18 10:33, Petr Jelinek wrote:
>>> Exactly.  The matview does not show up in pg_publication_tables but it's
>>> registered at some level.
>>
>> Indeed this is a bug. For normal publications we take care of this when
>> adding the relation to the publication but since ALL TABLES publications
>> don't check for membership we have to filter this directly in the output
>> plugin.
> 
> I think the filtering in pgoutput ought to make use of
> is_publishable_class() in some way.  That takes care of non-tables such
> as materialized views, but it also filters out the information_schema
> tables for example.  Right now, if you insert something into one of the
> IS tables, it gets shipped over the wire but is then dropped by the
> apply because there is no pg_subscription_rel entry of the table.  That
> doesn't quite have the user-visible effect as this bug, but it's bogus
> nonetheless.
> 
> So I propose this alternative patch that covers all these cases.

Committed, and also added a new test in the master branch.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services