pgsql: Remove the correct pgstat file on DROP DATABASE

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Remove the correct pgstat file on DROP DATABASE
Date: 2014-03-05 16:04:31
Message-ID: E1WLEJ5-0001pq-E8@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove the correct pgstat file on DROP DATABASE

We were unlinking the permanent file, not the non-permanent one. But
since the stat collector already unlinks all permanent files on startup,
there was nothing for it to unlink. The non-permanent file remained in
place, and was copied to the permanent directory on shutdown, so in
effect no file was ever dropped.

Backpatch to 9.3, where the issue was introduced by commit 187492b6c2e8.
Before that, there were no per-database files and thus no file to drop
on DROP DATABASE.

Per report from Thom Brown.

Author: Tomáš Vondra

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/13ea43ab83871a8bed5b9cdf7ec18c62bf63df4f

Modified Files
--------------
src/backend/postmaster/pgstat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2014-03-05 16:57:58 pgsql: Remove unused field "evttype".
Previous Message Bruce Momjian 2014-03-05 15:43:51 pgsql: pg_upgrade: improve C comment about what old/new oids match