From: | Rosser Schwarz <rosser(dot)schwarz(at)gmail(dot)com> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: Patch: add --if-exists to pg_recvlogical |
Date: | 2017-09-17 22:21:22 |
Message-ID: | CAFnxYwiOqt_5ke3y1RCEWxchQS_kW8Nd3XtSMJjOSi6ZinR5NA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Sep 1, 2017 at 10:22 AM, Peter Eisentraut <
peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> I understand the --drop-slot part. But I don't understand what it means
> to ignore a missing replication slot when running --start.
I'm not sure I do either, honestly. I followed the Principle of Least
Surprise in making it a no-op when those switches are used and the slot
doesn't exist, over "no one will ever do that". Because someone will.
I'm happy to hear suggestions on what to do in that case beyond exit
cleanly.
> The same option should be added to pg_receivewal as well.
Done.
On Fri, Sep 8, 2017 at 21:06 Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
wrote:
> Also "<option>--start<option>" breaks the documentation build
> (missing slash on the closing tag).
Fixed, thanks.
Please see revised patch, attached, addressing both comments.
rls
--
:wq
Attachment | Content-Type | Size |
---|---|---|
pg_recvlogical--if-exists-v2.patch | application/octet-stream | 8.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-09-17 22:52:05 | Re: [PATCH] Generic type subscripting |
Previous Message | Alvaro Herrera | 2017-09-17 21:38:35 | Re: SQL/JSON in PostgreSQL |