From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Dave Cramer <davecramer@postgres.rocks>
To: Michael Paquier <michael@paquier.xyz>
Cc: Masahiko Sawada <masahiko.sawada@2ndquadrant.com>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: Vladimir Sitnikov <sitnikov.vladimir@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: SIGSEGV from START_REPLICATION 0/XXXXXXX in XLogSendPhysical () at walsender.c:2762
Date: Wed, 3 Jun 2020 22:10:41 +0900
Message-ID: <3183662b-2e0b-4410-3bb8-4f4e4b029dd5@oss.nttdata.com> (raw)
In-Reply-To: <CADK3HHLWWHUpFZp1XAj3b4hTxZVUyaVJeooePck=UbQhPXTgQA@mail.gmail.com>
References: <CAB=Je-HnFcwPUeLq5CXUqBnC7G7WeTi7DdGKbnFe-EizvnBJmw@mail.gmail.com>
<20200528.181139.1631626601933187948.horikyota.ntt@gmail.com>
<20200529072138.GG44192@paquier.xyz>
<20200529.175653.1322451159722162490.horikyota.ntt@gmail.com>
<CA+fd4k76X533vzD4S3_y8gyXNzpNyV8SQndKJL6jxLPqVAB9+A@mail.gmail.com>
<20200602042456.GA3594@paquier.xyz>
<b7b1fcaa-2532-a1e9-6533-d74954608003@oss.nttdata.com>
<20200603051910.GG89559@paquier.xyz>
<CADK3HHLWWHUpFZp1XAj3b4hTxZVUyaVJeooePck=UbQhPXTgQA@mail.gmail.com>
On 2020/06/03 20:33, Dave Cramer wrote:
>
>
>
> On Wed, 3 Jun 2020 at 01:19, Michael Paquier <michael@paquier.xyz <mailto:michael@paquier.xyz>> wrote:
>
> On Tue, Jun 02, 2020 at 02:23:50PM +0900, Fujii Masao wrote:
> > On 2020/06/02 13:24, Michael Paquier wrote:
> >> Still unconvinced as this restriction stands for logical decoding
> >> requiring a database connection but it is not necessarily true now as
> >> physical replication has less restrictions than a logical one.
> >
> > Could you tell me what the benefit for supporting physical replication on
> > logical rep connection is? If it's only for "undocumented"
> > backward-compatibility, IMO it's better to reject such "tricky" set up.
> > But if there are some use cases for that, I'm ok to support that.
>
> Well, I don't really think that we can just break a behavior that
> exists since 9.4 as you could break applications relying on the
> existing behavior, and that's also the point of Vladimir upthread.
For the back branches, I agree with you. Even if it's undocumented behavior,
basically we should not get rid of it from the back branches unless there is
very special reason.
For v13, if it has no functional merit, I don't think it's so bad to get rid of
that undocumented (and maybe not-fully tested) behavior. If there are
applications depending it, I think that they can be updated.
> I don't see this is a valid reason to keep doing something. If it is broken then fix it.
> Clients can deal with the change.
+1
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
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: masao.fujii@oss.nttdata.com, davecramer@postgres.rocks, michael@paquier.xyz, masahiko.sawada@2ndquadrant.com, horikyota.ntt@gmail.com, sitnikov.vladimir@gmail.com
Subject: Re: SIGSEGV from START_REPLICATION 0/XXXXXXX in XLogSendPhysical () at walsender.c:2762
In-Reply-To: <3183662b-2e0b-4410-3bb8-4f4e4b029dd5@oss.nttdata.com>
* 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