09.08.2025 22:54, Kirill Reshke пишет: > On Thu, 7 Aug 2025 at 21:36, Aleksander Alekseev > <aleksan...@tigerdata.com> wrote: > >> Perhaps there was a good >> reason to update the VM *before* creating WAL records I'm unaware of. > > Looks like 503c730 intentionally does it this way; however, I have not > yet fully understood the reasoning behind it.
I repeat: there was no intention. Neither in commit message, nor in discussion about. There was intention to move visibilitymap_clear under heap page lock and into critical section, but there were no any word about logging. I believe, it was just an unfortunate oversight that the change is made before logging. -- regards Yura Sokolov aka funny-falcon
