pgsql: Fix wrong data table horizon computation during backend startup. - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: Fix wrong data table horizon computation during backend startup.
Date
Msg-id E1kXzx9-0006ie-RW@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix wrong data table horizon computation during backend startup.

When ComputeXidHorizons() was called before MyDatabaseOid is set,
e.g. because a dead row in a shared relation is encountered during
InitPostgres(), the horizon for normal tables was computed too
aggressively, ignoring all backends connected to a database.

During subsequent pruning in a data table the too aggressive horizon
could end up still being used, possibly leading to still needed tuples
being removed. Not good.

This is a bug in dc7420c2c92, which the test added in 94bc27b5768 made
visible, if run with force_parallel_mode set to regress. In that case
the bug is reliably triggered, because "pruning_query" is run in a
parallel worker and the start of that parallel worker is likely to
encounter a dead row in pg_database.

The fix is trivial: Compute a more pessimistic data table horizon if
MyDatabaseId is not yet known.

Author: Andres Freund
Discussion: https://postgr.es/m/20201029040030.p4osrmaywhqaesd4@alap3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1c7675a7a4265064a2c8e1ed02b2c042c2521664

Modified Files
--------------
src/backend/storage/ipc/procarray.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: pgsql: Centralize horizon determination for temp tables, fixing bug due
Next
From: Tom Lane
Date:
Subject: pgsql: Don't use custom OID symbols in pg_type.dat, either.