> On 6 Mar 2026, at 00:27, Andres Freund <[email protected]> wrote:
> 
> No, that'd be a completely bogus LSN, as CurrBytePos does not include any
> space for page headers, to make the the very contended spinlock'ed section in
> ReserveXLogInsertLocation() cheaper

Thanks! 
Now I see it's documented that "usable bytes position" is not XLogRecPtr at all.
And XLogBytePosToEndRecPtr() is unavoidable for newly created relations in
wal_level=minimal.

Perhaps we can even add an Assert that PageSetLSN() does not stamp LSNs from 
future.
Sorry for raising off-topic questions, I'll go check this myself.


Best regards, Andrey Borodin.

Reply via email to