Re: A few patches to clarify snapshot management

2025-03-13 Thread Heikki Linnakangas
On 06/01/2025 23:30, Heikki Linnakangas wrote: On 20/12/2024 19:31, Heikki Linnakangas wrote: /*  * Struct representing all kind of possible snapshots.  *  * There are several different kinds of snapshots:  * * Normal MVCC snapshots  * * MVCC snapshots taken during recovery (in Hot-Standby mode)

Re: A few patches to clarify snapshot management

2025-01-07 Thread Heikki Linnakangas
On 07/01/2025 00:00, Andres Freund wrote: On 2024-12-20 19:31:01 +0200, Heikki Linnakangas wrote: While playing around some more with this, I noticed that this code in GetTransactionSnapshot() is never reached, and AFAICS has always been dead code: Snapshot GetTransactionSnapshot(void) {

Re: A few patches to clarify snapshot management

2025-01-06 Thread Andres Freund
On 2024-12-20 19:31:01 +0200, Heikki Linnakangas wrote: > On 16/12/2024 23:56, Nathan Bossart wrote: > > On Mon, Dec 16, 2024 at 12:06:33PM +0200, Heikki Linnakangas wrote: > > > While working on the CSN snapshot patch, I got sidetracked looking closer > > > into the snapshot tracking in snapmgr.c.

Re: A few patches to clarify snapshot management

2024-12-20 Thread Heikki Linnakangas
On 16/12/2024 23:56, Nathan Bossart wrote: On Mon, Dec 16, 2024 at 12:06:33PM +0200, Heikki Linnakangas wrote: While working on the CSN snapshot patch, I got sidetracked looking closer into the snapshot tracking in snapmgr.c. Attached are a few patches to clarify some things. I haven't yet loo

Re: A few patches to clarify snapshot management

2024-12-16 Thread Nathan Bossart
On Mon, Dec 16, 2024 at 12:06:33PM +0200, Heikki Linnakangas wrote: > While working on the CSN snapshot patch, I got sidetracked looking closer > into the snapshot tracking in snapmgr.c. Attached are a few patches to > clarify some things. I haven't yet looked closely at what you are proposing, bu

A few patches to clarify snapshot management

2024-12-16 Thread Heikki Linnakangas
While working on the CSN snapshot patch, I got sidetracked looking closer into the snapshot tracking in snapmgr.c. Attached are a few patches to clarify some things. # Patch 1: Remove unnecessary GetTransactionSnapshot() calls and FIXME comments In commit dc7420c2c927, Andres added FIXME com