From: | Petr Jelinek <petr(at)2ndquadrant(dot)com> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | Steve Singer <steve(at)ssinger(dot)info>, Craig Ringer <craig(at)2ndquadrant(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Logical Replication WIP |
Date: | 2016-11-11 11:15:46 |
Message-ID: | 8287bc28-d0ea-c0ac-0e8f-b4768f51c63c@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 08/11/16 19:51, Peter Eisentraut wrote:
> Review of v7 0003-Add-PUBLICATION-catalogs-and-DDL.patch:
>
> This appears to address previous reviews and is looking pretty solid. I
> have some comments that are easily addressed:
>
> [still from previous review] The code for OCLASS_PUBLICATION_REL in
> getObjectIdentityParts() does not fill in objname and objargs, as it is
> supposed to.
>
> catalog.sgml: pg_publication_rel column names must be updated after renaming
>
> alter_publication.sgml and elsewhere: typos PuBLISH_INSERT etc.
>
> create_publication.sgml: FOR TABLE ALL IN SCHEMA does not exist anymore
>
> create_publication.sgml: talks about not-yet-existing SUBSCRIPTION role
>
> DropPublicationById maybe name RemovePublicationById for consistency
>
> system_views.sql: C.relkind = 'r' unnecessary
>
> CheckCmdReplicaIdentity: error message says "cannot update", should
> distinguish between update and delete
>
> relcache.c: pubactions->pubinsert |= pubform->pubinsert; etc. should be ||=
>
> RelationData.rd_pubactions could be a bitmap, simplifying some memcpy
> and context management. But RelationData appears to favor rich data
> structures, so maybe that is fine.
>
Thanks for these, some of it is result of various rebases that I did
(the sync patch makes rebasing bit complicated as it touches everything)
and it's easy for me to overlook it at this point.
--
Petr Jelinek http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Etsuro Fujita | 2016-11-11 11:30:39 | Re: Push down more UPDATEs/DELETEs in postgres_fdw |
Previous Message | Petr Jelinek | 2016-11-11 11:12:36 | Re: Logical Replication WIP |