On 25/2/26 08:04, vignesh C wrote: > On Mon, 23 Feb 2026 at 16:46, Amit Kapila <[email protected]> wrote: > The attached patch has the changes for the same i.e.a) Raises an error > when attempting to attach a partition to a root partitioned table if > that table is referenced in an EXCEPT clause of any publication. b) > Adds support for dropping excluded tables using: ALTER PUBLICATION ... > DROP EXCEPT TABLE. c) Adds support for replacing the exclusion list > using ALTER PUBLICATION ... SET EXCEPT TABLE. > The changes related to DROP EXCEPT TABLE and SET EXCEPT TABLE have > been kept separately into patch 0002 for easier review.
I discovered this patch, maybe not deeply enough. But one question raised. I usually work with multiple tables (sometimes hundreds, if not thousands). EXCEPT clause might be quite rare. Some commands want to extract only excepted tables from the publication using the following pattern: 'SELECT ... FROM pg_publication_rel WHERE prpubid = <X> AND pr.prexcept' - check_publications_except_list() - describePublications() - getPublications() - LoadPublications() / publication_has_any_except_table() That means pass through all the publication tables to find (usually) nothing. It might be a costly operation, especially in the LoadPublications routine. Maybe slightly change index pg_publication_rel_prpubid_index a little and add 'prexcept' as a second column there? It might avoid performance issues that may arise during the upgrade. -- regards, Andrei Lepikhov, pgEdge
