From: | Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com>, Rahila Syed <rahilasyed90(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Column Filtering in Logical Replication |
Date: | 2022-03-02 12:13:15 |
Message-ID: | cac80284-7196-6c77-adb9-ef0976a3e21f@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
Attached is an updated patch, addressing most of the issues reported so
far. There are various minor tweaks, but the main changes are:
1) regular regression tests, verifying (hopefully) all the various cases
of publication vs. column filters, replica identity check at various
changes and so on
2) pg_dump tests, testing column filters (alone and with row filter)
3) checks of column filter vs. publish_via_partition_root and replica
identity, following the same logic as the row-filter patch (hopefully,
it touches the same places, using the same logic, ...)
That means - with "publish_via_partition_root=false" it's not allowed to
specify column filters on partitioned tables, only for leaf partitions.
And we check column filter vs. replica identity when adding tables to
publications, or whenever we change the replica identity.
The patch is still a bit crude, I'm sure some of the places (especially
the new ones) may need cleanup/recovery. But I think it's much closer to
being committable, I think.
The first two simple patches are adding tests for the row filtering. So
this is not really part of this patch.
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
0001-Verify-changing-WHERE-condition-for-a-publi-20220302.patch | text/x-patch | 6.9 KB |
0002-Test-publication-row-filters-in-pg_dump-tes-20220302.patch | text/x-patch | 1.3 KB |
0003-Allow-specifying-column-filters-for-logical-20220302.patch | text/x-patch | 140.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bharath Rupireddy | 2022-03-02 12:48:10 | Re: Add CHECKPOINT_REQUESTED flag to the log message in LogCheckpointStart() |
Previous Message | Euler Taveira | 2022-03-02 12:11:47 | Re: row filtering for logical replication |