From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Antonin Houska <ah(at)cybertec(dot)at> |
Cc: | Arseny Sher <a(dot)sher(at)postgrespro(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Fix slot's xmin advancement and subxact's lost snapshots in decoding. |
Date: | 2018-06-18 18:04:19 |
Message-ID: | 20180618180419.mysc6nxvntgvfgg2@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2018-Jun-11, Antonin Houska wrote:
> Arseny Sher <a(dot)sher(at)postgrespro(dot)ru> wrote:
> > Please see detailed description of the issues, tests which reproduce
> > them and fixes in the attached patch.
>
> I've played with your tests and gdb for a while, both w/o and with your
> patch. I think I can understand both problems. I could not invent simpler way
> to fix them.
>
> One comment about the coding: since you introduce a new transaction list and
> it's sorted by LSN, I think you should make the function AssertTXNLsnOrder()
> more generic and use it to check the by_base_snapshot_lsn list too. For
> example call it after the list insertion and deletion in
> ReorderBufferPassBaseSnapshot().
I've been going over this patch also, and I've made a few minor edits of
the patch and the existing code as I come to understand what it's all
about.
> Also I think it makes no harm if you split the diff into two, although both
> fixes use the by_base_snapshot_lsn field.
Please don't.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Fujii Masao | 2018-06-18 18:06:54 | Re: Speedup of relation deletes during recovery |
Previous Message | Robert Haas | 2018-06-18 17:43:57 | Re: Removing "Included attributes in B-tree indexes" section from docs |