]> granicus.if.org Git - postgresql/commit
Remove the pgstat_drop_relation() call from smgr_internal_unlink(), because
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 8 Jul 2007 22:23:32 +0000 (22:23 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 8 Jul 2007 22:23:32 +0000 (22:23 +0000)
commit4ceaf35ff3ce7ab91aed0639882690e777a569d4
tree95d44eab6a38f4c539b652e27fee791ac29a9442
parent5d798ba82fb92fac7c75947df3f5059bebd67a1f
Remove the pgstat_drop_relation() call from smgr_internal_unlink(), because
we don't know at that point which relation OID to tell pgstat to forget.
The code was passing the relfilenode, which is incorrect, and could possibly
cause some other relation's stats to be zeroed out.  While we could try to
clean this up, it seems much simpler and more reliable to let the next
invocation of pgstat_vacuum_tabstat() fix things; which indeed is how it
worked before I introduced the buggy code into 8.1.3 and later :-(.
Problem noticed by Itagaki Takahiro, fix is per subsequent discussion.
src/backend/postmaster/pgstat.c
src/backend/storage/smgr/smgr.c
src/include/pgstat.h