pg.ddx.io pgsql-committers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
pgsql: Revert "Test that vacuum removes tuples older than OldestXmin" 2+ messages / 1 participants [nested] [flat]
* pgsql: Revert "Test that vacuum removes tuples older than OldestXmin" @ 2024-07-22 21:01 Melanie Plageman <melanieplageman@gmail.com> 0 siblings, 0 replies; 2+ messages in thread From: Melanie Plageman @ 2024-07-22 21:01 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Revert "Test that vacuum removes tuples older than OldestXmin" This reverts commit aa607980aee08416211f003ab41aa750f5559712. This test proved to be unstable on the buildfarm, timing out before the standby could catch up on 32-bit machines where more rows were required and failing to reliably trigger multiple index vacuum rounds on 64-bit machines where fewer rows should be required. Because the instability is only known to be present on versions of Postgres with TIDStore used for dead TID storage by vacuum, this is only being reverted on master and REL_17_STABLE. As having this coverage may be valuable, there is a discussion on the thread of possible ways to stabilize the test. If that happens, a fixed test can be committed again. Backpatch-through: 17 Reported-by: Tom Lane Discussion: https://postgr.es/m/614152.1721580711%40sss.pgh.pa.us Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/efcbb76efe406d59c2ba8b4a09e04c01158ba575 Modified Files -------------- src/test/recovery/meson.build | 1 - src/test/recovery/t/043_vacuum_horizon_floor.pl | 268 ------------------------ 2 files changed, 269 deletions(-) ^ permalink raw reply [nested|flat] 2+ messages in thread
* pgsql: Revert "Test that vacuum removes tuples older than OldestXmin" @ 2024-07-22 21:01 Melanie Plageman <melanieplageman@gmail.com> 0 siblings, 0 replies; 2+ messages in thread From: Melanie Plageman @ 2024-07-22 21:01 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Revert "Test that vacuum removes tuples older than OldestXmin" This reverts commit 80c34692e8e674e3b2f150f248ef2002ae2ac3a7. This test proved to be unstable on the buildfarm, timing out before the standby could catch up on 32-bit machines where more rows were required and failing to reliably trigger multiple index vacuum rounds on 64-bit machines where fewer rows should be required. Because the instability is only known to be present on versions of Postgres with TIDStore used for dead TID storage by vacuum, this is only being reverted on master and REL_17_STABLE. As having this coverage may be valuable, there is a discussion on the thread of possible ways to stabilize the test. If that happens, a fixed test can be committed again. Backpatch-through: 17 Reported-by: Tom Lane Discussion: https://postgr.es/m/614152.1721580711%40sss.pgh.pa.us Branch ------ REL_17_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/1a3e90948b50abef3a075c5b1d1c59a9d631a187 Modified Files -------------- src/test/recovery/meson.build | 1 - src/test/recovery/t/043_vacuum_horizon_floor.pl | 268 ------------------------ 2 files changed, 269 deletions(-) ^ permalink raw reply [nested|flat] 2+ messages in thread
end of thread, other threads:[~2024-07-22 21:01 UTC | newest] Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2024-07-22 21:01 pgsql: Revert "Test that vacuum removes tuples older than OldestXmin" Melanie Plageman <melanieplageman@gmail.com> 2024-07-22 21:01 pgsql: Revert "Test that vacuum removes tuples older than OldestXmin" Melanie Plageman <melanieplageman@gmail.com>
This inbox is served by DDX for PostgreSQL; see mirroring instructions for how to clone and mirror all data and code used for this inbox