From: Tom Lane <tgl@sss.pgh.pa.us>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Petr Jelinek <petr.jelinek@2ndquadrant.com>
Cc: Michael Paquier <michael.paquier@gmail.com>
Cc: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: logical replication syntax (was DROP SUBSCRIPTION, query cancellations and slot handling)
Date: Thu, 04 May 2017 17:29:39 -0400
Message-ID: <30885.1493933379@sss.pgh.pa.us> (raw)
In-Reply-To: <CA+Tgmoba_-AWw9QxeDiEBaGfB8Whaukba4GEm=Jv8yKarF_F0g@mail.gmail.com>
References: <CA+TgmoZmkbpAWRzVKDVcHnTBkYjJEFS8=09RL-G3zgdozCLFHQ@mail.gmail.com>
<27478.1493727233@sss.pgh.pa.us>
<cacf7538-5658-a93f-c03a-83d309ed1d9a@2ndquadrant.com>
<CA+TgmoYe8_FZ9oNTxL-q42kUXGKCDrrG512qbyKAQw9ROBvQ9g@mail.gmail.com>
<0afdeda8-d8a1-56d8-f637-7f110c2ca443@2ndquadrant.com>
<CA+TgmoZbPy9ufRQE46HGA-2u3Cj9Z_vHtkvtpW9N2pW4LmGQ_Q@mail.gmail.com>
<647cb9ea-4b9b-75d3-df12-c52ddfd512a8@2ndquadrant.com>
<CA+Tgmoba_-AWw9QxeDiEBaGfB8Whaukba4GEm=Jv8yKarF_F0g@mail.gmail.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Robert Haas <robertmhaas@gmail.com> writes:
> On Wed, May 3, 2017 at 12:38 AM, Petr Jelinek
> <petr.jelinek@2ndquadrant.com> wrote:
>> Ok, Let me be clear, I actually happen to agree with your proposal. The
>> reason I am moaning is that I always seem to find myself doing tons of
>> mechanical work to rewrite some cosmetic aspect of some patch based on
>> which committer is paying attention in a specific week. So while I am
>> for doing exactly what you proposed, I'd like to see couple of +1s first
>> (Peter?) since I don't want to rewrite it to something different again
>> and it's also long past freeze.
> So, Tom Lane and Thom Brown and Josh Drake all seemed generally in
> favor of cleaning this up. Perhaps they could opine on this
> particular proposal.
It seems like there's some remaining indecision between "make it look
like the options in EXPLAIN, VACUUM, etc" and "make it look like the
WITH options found in some other statements". I do not have a strong
opinion which one to do, but I'd definitely say that you should use WITH
in the latter case but not in the former. I think this mostly boils down
to whether to use "=" or not; you've got "not" in the proposal, which
means you are following the EXPLAIN precedent and should not use WITH.
I'm okay with the other specifics mentioned.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, robertmhaas@gmail.com, petr.jelinek@2ndquadrant.com, michael.paquier@gmail.com, peter.eisentraut@2ndquadrant.com
Subject: Re: logical replication syntax (was DROP SUBSCRIPTION, query cancellations and slot handling)
In-Reply-To: <30885.1493933379@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox