|From:||Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>|
|Cc:||andres(at)anarazel(dot)de, nag1010(at)gmail(dot)com, jdnelson(at)dyn(dot)com, pgsql-hackers(at)postgresql(dot)org|
|Subject:||Re: [BUGS] Bug in Physical Replication Slots (at least 9.5)?|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
At Mon, 4 Sep 2017 17:17:19 +0900, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote in <CAB7nPqSPf0qkq=DhSO-tAM9++LSA2aEYSVJ3oY_EdUdb=jKi1w(at)mail(dot)gmail(dot)com>
> On Mon, Sep 4, 2017 at 4:04 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
> > I've not read through the thread, but this seems like the wrong approach
> > to me. The receiving side should use a correct value, instead of putting
> > this complexity on the sender's side.
> Yes I agree with that. The current patch gives me a bad feeling to be
> honest with the way it does things..
The problem is that the current ReadRecord needs the first one of
a series of continuation records from the same source with the
other part, the master in the case.
A (or the) solution closed in the standby side is allowing to
read a seris of continuation records from muliple sources. In
this case the first part from the standby's pg_wal and the second
part from the master via streaming replication. ReadRecord
needed refactoring, (seems to me) breaking the concept of
XLogReader plug-in system to accomplish this behavior.
If it is preferable for you, I'll re-try that. Or hints for other
solutions are also welcome.
Is there any suggestions?
NTT Open Source Software Center
|Next Message||Alvaro Herrera||2017-09-06 10:40:20||Re: Old row version in hot chain become visible after a freeze|
|Previous Message||Michael Paquier||2017-09-06 04:02:57||Re: Old row version in hot chain become visible after a freeze|
|Next Message||Alvaro Herrera||2017-09-06 09:07:29||Re: Replacing lfirst() with lfirst_node() appropriately in planner.c|
|Previous Message||Fabien COELHO||2017-09-06 08:04:40||Re: psql - add special variable to reflect the last query status|