diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-11-21 10:50:50 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-11-21 10:50:50 -0500 |
commit | e2933a6e11791191050cd925d52d34e785eece77 (patch) | |
tree | ed085f22f14202434f078c559a66fd43a2c81afa /contrib/pageinspect/expected | |
parent | f193883fc9cebe8fa20359b0797832837a788112 (diff) | |
download | postgresql-e2933a6e11791191050cd925d52d34e785eece77.tar.gz |
Prevent instability in contrib/pageinspect's regression test.
pageinspect has occasionally failed on slow buildfarm members,
with symptoms indicating that the expected effects of VACUUM
FREEZE didn't happen. This is presumably because a background
transaction such as auto-analyze was holding back global xmin.
We can work around that by using a temp table in the test.
Since commit a7212be8b, that will use an up-to-date cutoff xmin
regardless of other processes. And pageinspect itself shouldn't
really care whether the table is temp.
Back-patch to v14. There would be no point in older branches
without back-patching a7212be8b, which seems like more trouble
than the problem is worth.
Discussion: https://postgr.es/m/2892135.1668976646@sss.pgh.pa.us
Diffstat (limited to 'contrib/pageinspect/expected')
-rw-r--r-- | contrib/pageinspect/expected/page.out | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/contrib/pageinspect/expected/page.out b/contrib/pageinspect/expected/page.out index 3bdc37bbf5..80ddb45a60 100644 --- a/contrib/pageinspect/expected/page.out +++ b/contrib/pageinspect/expected/page.out @@ -1,5 +1,6 @@ CREATE EXTENSION pageinspect; -CREATE TABLE test1 (a int, b int); +-- Use a temp table so that effects of VACUUM are predictable +CREATE TEMP TABLE test1 (a int, b int); INSERT INTO test1 VALUES (16777217, 131584); VACUUM (DISABLE_PAGE_SKIPPING) test1; -- set up FSM -- The page contents can vary, so just test that it can be read |