> I revisited the CheckPointSnapBuild() and > CheckPointLogicalRewriteHeap(). Both functions work idempotently and > are safe to move after CheckPointReplicationSlots(). They benefit from > the updated last_saved_restart_lsn computed in the current checkpoint > cycle, enabling more accurate cleanup decisions. > > I ran regression testing comparing the baseline (master) version with > the patched version: > > - Checkpoint duration: No change. > - WAL retention: More accurate, because it uses the current > replication slot state. > - Regressions: None detected. > - Dependencies: No new dependencies introduced. > > Please see the v2 as attached but we can also commit the v1 since the > only difference is the order of CheckPointSnapBuild and > CheckPointLogicalRewriteHeap working idempotently.
Rebased it against the head. Regards, Demir.
v3-0001-Checkpoint-replication-slots-late-in-the-cycle.patch
Description: Binary data
