From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Erik Rijkers <er(at)xs4all(dot)nl>, Steve Singer <steve(at)ssinger(dot)info>, Craig Ringer <craig(at)2ndquadrant(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Logical Replication WIP |
Date: | 2017-01-04 17:46:36 |
Message-ID: | e12a5167-025e-c698-35ca-e0272a53e5d3@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Some small patches for 0002-Add-SUBSCRIPTION-catalog-and-DDL-v16.patch.gz:
- Add a get_subscription_name() function
- Remove call for ApplyLauncherWakeupAtCommit() (rebasing error?)
- Remove some unused include files (same as before)
- Rename pg_dump --no-create-subscription-slot to
--no-create-subscription-slots (plural), add documentation.
In CreateSubscription(), I don't think we should connect to the remote
if no slot creation is requested. Arguably, the point of that option is
to not make network connections. (That is what my documentation patch
above claims, in any case.)
I don't know why we need to check the PostgreSQL version number of the
remote. We should rely on the protocol version number, and we should
just make it work. When PG 11 comes around, subscribing from PG 10 to a
publisher on PG 11 should just work without any warnings, IMO.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Attachment | Content-Type | Size |
---|---|---|
0001-fixup-Add-SUBSCRIPTION-catalog-and-DDL.patch | text/x-patch | 3.2 KB |
0002-fixup-Add-SUBSCRIPTION-catalog-and-DDL.patch | text/x-patch | 776 bytes |
0003-fixup-Add-SUBSCRIPTION-catalog-and-DDL.patch | text/x-patch | 2.5 KB |
0004-fixup-Add-SUBSCRIPTION-catalog-and-DDL.patch | text/x-patch | 4.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2017-01-04 17:49:56 | Re: proposal: session server side variables |
Previous Message | Pavel Stehule | 2017-01-04 17:34:01 | Re: proposal: session server side variables |