On Mon, Dec 10, 2018 at 4:58 PM Andrey Borodin <x4...@yandex-team.ru> wrote: > > 10 дек. 2018 г., в 18:56, Andrey Borodin <x4...@yandex-team.ru> написал(а): > > > > PFA this part. In thread about GiST vacuum Heikki was not very happy with > > reusing PageHeader->pd_prune_xid for this. But we somewhat concluded that > > this might be OK. > > Anyway, there's whole page, we can store those bits anywhere.
We might allocate special field in opaque to store stat xid like B-tree or SP-GiST. But we have to be binary compatible. And this change shouldn't require upgrade procedure, since we're going to backpatch that. We could allocate space elsewhere on the page, but that would be a bit cumbersome. So, let's use pd_prune_xid once it's available. > Errrmm... I'm very sorry for the noise. I've forgot the actual patch. Attached patch appears to be incomplete. GinPageSetDeleteXid() is called only in ginRedoDeletePage(), so only in recovery, while it should be set during normal work too. deleteXid field of ginxlogDeletePage is never set. ------ Alexander Korotkov Postgres Professional: http://www.postgrespro.com The Russian Postgres Company