Re: nbtree VACUUM's REDO routine doesn't clear page's VACUUM cycle ID

2024-12-23 Thread Peter Geoghegan
On Wed, Nov 20, 2024 at 4:41 AM Andrey M. Borodin wrote: > > On 15 Nov 2024, at 21:33, Peter Geoghegan wrote: > > I propose this for the master branch only. > > The change seems correct to me: anyway cycle must be less than cycle of any > future vacuum after promotion. The cycles set in the pag

Re: nbtree VACUUM's REDO routine doesn't clear page's VACUUM cycle ID

2024-12-23 Thread Peter Geoghegan
On Wed, Nov 20, 2024 at 1:32 AM Michael Paquier wrote: > Perhaps this patch should also explain why this is better this way? See the commit message, and my remarks to Andrey just now. > This path does not manipulate btpo_level, for one. Why would it do so? Changing the btree level would corrupt

Re: nbtree VACUUM's REDO routine doesn't clear page's VACUUM cycle ID

2024-11-20 Thread Andrey M. Borodin
> On 15 Nov 2024, at 21:33, Peter Geoghegan wrote: > > Attached patch teaches btree_xlog_vacuum, nbtree VACUUM's REDO > routine, to reset the target page's opaque->btpo_cycleid to 0. This > makes the REDO routine match original execution, which seems like a > good idea on consistency grounds.

Re: nbtree VACUUM's REDO routine doesn't clear page's VACUUM cycle ID

2024-11-19 Thread Michael Paquier
On Fri, Nov 15, 2024 at 11:33:37AM -0500, Peter Geoghegan wrote: > Attached patch teaches btree_xlog_vacuum, nbtree VACUUM's REDO > routine, to reset the target page's opaque->btpo_cycleid to 0. This > makes the REDO routine match original execution, which seems like a > good idea on consistency gr

nbtree VACUUM's REDO routine doesn't clear page's VACUUM cycle ID

2024-11-15 Thread Peter Geoghegan
Attached patch teaches btree_xlog_vacuum, nbtree VACUUM's REDO routine, to reset the target page's opaque->btpo_cycleid to 0. This makes the REDO routine match original execution, which seems like a good idea on consistency grounds. I propose this for the master branch only. -- Peter Geoghegan