From 8277be755b187e66a57bfff15a2d46f98656f4ca Mon Sep 17 00:00:00 2001 From: Melanie Plageman Date: Thu, 31 Jul 2025 14:07:51 -0400 Subject: [PATCH v12 10/20] Rename PruneState.freeze to attempt_freeze This makes it more clear that this is to indicate the caller would like heap_page_prune_and_freeze() to consider freezing tuples -- not that we ultimately will end up freezing them. Also rename local variable hint_bit_fpi to did_tuple_hint_fpi. This makes it clear it is about tuple hints and not page hints and that it indicates something that happened and not something that could happen. --- src/backend/access/heap/pruneheap.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/backend/access/heap/pruneheap.c b/src/backend/access/heap/pruneheap.c index 398962ed1cb..df3e6439176 100644 --- a/src/backend/access/heap/pruneheap.c +++ b/src/backend/access/heap/pruneheap.c @@ -43,7 +43,7 @@ typedef struct /* whether or not dead items can be set LP_UNUSED during pruning */ bool mark_unused_now; /* whether to attempt freezing tuples */ - bool freeze; + bool attempt_freeze; struct VacuumCutoffs *cutoffs; /*------------------------------------------------------- @@ -445,13 +445,13 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, bool do_hint; uint8 vmflags = 0; uint8 old_vmbits = 0; - bool hint_bit_fpi; + bool did_tuple_hint_fpi; int64 fpi_before = pgWalUsage.wal_fpi; /* Copy parameters to prstate */ prstate.vistest = vistest; prstate.mark_unused_now = (options & HEAP_PAGE_PRUNE_MARK_UNUSED_NOW) != 0; - prstate.freeze = (options & HEAP_PAGE_PRUNE_FREEZE) != 0; + prstate.attempt_freeze = (options & HEAP_PAGE_PRUNE_FREEZE) != 0; prstate.cutoffs = cutoffs; /* @@ -473,7 +473,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, /* initialize page freezing working state */ prstate.pagefrz.freeze_required = false; - if (prstate.freeze) + if (prstate.attempt_freeze) { Assert(new_relfrozen_xid && new_relmin_mxid); prstate.pagefrz.FreezePageRelfrozenXid = *new_relfrozen_xid; @@ -520,7 +520,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, * function, when we return the value to the caller, so that the caller * doesn't set the VM bit incorrectly. */ - if (prstate.freeze) + if (prstate.attempt_freeze) { prstate.all_visible = true; prstate.all_frozen = true; @@ -634,7 +634,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, * If checksums are enabled, heap_prune_satisfies_vacuum() may have caused * an FPI to be emitted. */ - hint_bit_fpi = fpi_before != pgWalUsage.wal_fpi; + did_tuple_hint_fpi = fpi_before != pgWalUsage.wal_fpi; /* * Process HOT chains. @@ -750,7 +750,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, * plans we prepared, or not. */ do_freeze = false; - if (prstate.freeze) + if (prstate.attempt_freeze) { if (prstate.pagefrz.freeze_required) { @@ -783,7 +783,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, */ if (RelationNeedsWAL(relation)) { - if (hint_bit_fpi) + if (did_tuple_hint_fpi) do_freeze = true; else if (do_prune) { @@ -1046,7 +1046,7 @@ heap_page_prune_and_freeze(Relation relation, Buffer buffer, presult->old_vmbits = old_vmbits; presult->new_vmbits = vmflags; - if (prstate.freeze) + if (prstate.attempt_freeze) { if (presult->nfrozen > 0) { @@ -1628,7 +1628,7 @@ heap_prune_record_unchanged_lp_normal(Page page, PruneState *prstate, OffsetNumb } /* Consider freezing any normal tuples which will not be removed */ - if (prstate->freeze) + if (prstate->attempt_freeze) { bool totally_frozen; -- 2.43.0