Re: Hash index AM fake LSN oversight

On Wed, Jul 15, 2026 at 2:47 AM Ewan Young  wrote:
> Confirmed -- log_split_page() is the only data-page site e5836f7b left
> without a fake-LSN branch (the _hash_init meta/bitmap paths set the LSN
> under WAL only, but those aren't read concurrently, so that's expected).
> The fix is correct.

Pushed. Thanks for the review!

> One optional cosmetic point: the other fake-LSN sites, including the two
> functions e5836f7b just reworked in this file, hoist recptr out of the
> if and call PageSetLSN once afterwards, rather than once per branch.

I agree that it works slightly better that way; that's what I did in
the committed version.

--
Peter Geoghegan