database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-21 12:05:57 +03:00
Code

Suppress attempts to report dropped tables to the stats collector from a

Browse Source 
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.
This commit is contained in:
Tom Lane 2006-03-30 22:11:59 +00:00
parent c119cdcab0
commit 21b54ed81a
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/backend/storage/smgr/smgr.c
View File

@ -11,7 +11,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/storage/smgr/smgr.c,v 1.93.2.2 2006/01/18 20:35:16 tgl Exp $ * $PostgreSQL: pgsql/src/backend/storage/smgr/smgr.c,v 1.93.2.3 2006/03/30 22:11:59 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -470,8 +470,14 @@ smgr_internal_unlink(RelFileNode rnode, int which, bool isTemp, bool isRedo)
*/ */
FreeSpaceMapForgetRel(&rnode); 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, pgstats.c will get confused because we aren't a real
* backend process).
*/
if (!InRecovery)
pgstat_drop_relation(rnode.relNode);
/* /*
* And delete the physical files. * And delete the physical files.