| From: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | alvherre(at)2ndquadrant(dot)com |
| Cc: | michael(at)paquier(dot)xyz, simon(at)2ndquadrant(dot)com, a(dot)sher(at)postgrespro(dot)ru, sawada(dot)mshk(at)gmail(dot)com, k(dot)knizhnik(at)postgrespro(dot)ru, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Possible bug in logical replication. |
| Date: | 2018-06-22 07:33:12 |
| Message-ID: | 20180622.163312.254556300.horiguchi.kyotaro@lab.ntt.co.jp |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello.
At Thu, 14 Jun 2018 16:06:43 -0400, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote in <20180614200643(dot)3my362zmfiwitrni(at)alvherre(dot)pgsql>
> Can somebody (Arseny, Konstantin, horiguti, Sawada) please confirm that
> Michaël's commit fixes the reported bug?
pg_advance_replication_slots can advance uninitialized physical
slots and that might not be good. (Logical slots always have
initial invalid values in thw two lsn columns.)
About scanning from restart_lsn in the advancing function, I
think I confirmed that the value always comes from
XLogRecordBuffer.origptr, which comes from ReadRecPtr, not
EndRecPtr, which cannot be on page boundary.
FWIW, as the result, it looks fine for me also regarding the
issue on this thread.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
| From | Date | Subject | |
|---|---|---|---|
| Next Message | shao bret | 2018-06-22 07:34:21 | 答复: Incorrect comments in commit_ts.c |
| Previous Message | Ashutosh Sharma | 2018-06-22 07:30:45 | Re: Incorrect errno used with %m for backend code |