From 4243f2387a762b26c8e6be5c8094425798d86ea3 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Thu, 30 Mar 2006 22:11:55 +0000 Subject: [PATCH] Suppress attempts to report dropped tables to the stats collector from a startup or recovery process. Since such a process isn't a real backend, pgstat.c gets confused. This accounts for recent reports of strange "invalid server process ID -1" log messages during crash recovery. There isn't any point in attempting to make the report, since we'll discard stats in such scenarios anyhow. --- src/backend/storage/smgr/smgr.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/backend/storage/smgr/smgr.c b/src/backend/storage/smgr/smgr.c index 2ab1268ba1..ef7c3994fe 100644 --- a/src/backend/storage/smgr/smgr.c +++ b/src/backend/storage/smgr/smgr.c @@ -11,7 +11,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/storage/smgr/smgr.c,v 1.97 2006/03/24 04:32:13 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/storage/smgr/smgr.c,v 1.98 2006/03/30 22:11:55 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -470,8 +470,14 @@ smgr_internal_unlink(RelFileNode rnode, int which, bool isTemp, bool isRedo) */ FreeSpaceMapForgetRel(&rnode); - /* Tell the stats collector to forget it immediately, too. */ - pgstat_drop_relation(rnode.relNode); + /* + * Tell the stats collector to forget it immediately, too. Skip this + * in recovery mode, since the stats collector likely isn't running + * (and if it is, pgstat.c will get confused because we aren't a real + * backend process). + */ + if (!InRecovery) + pgstat_drop_relation(rnode.relNode); /* * And delete the physical files. -- 2.11.0