From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Add CheckTableNotInUse calls in DROP TABLE and DROP INDEX. |
Date: | 2011-02-15 20:51:07 |
Message-ID: | E1PpRrX-0007hj-DE@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Add CheckTableNotInUse calls in DROP TABLE and DROP INDEX.
Recent releases had a check on rel->rd_refcnt in heap_drop_with_catalog,
but failed to cover the possibility of pending trigger events at DROP time.
(Before 8.4 we didn't even check the refcnt.) When the trigger events were
eventually fired, you'd get "could not open relation with OID nnn" errors,
as in recent report from strk. Better to throw a suitable error when the
DROP is attempted.
Also add a similar check in DROP INDEX.
Back-patch to all supported branches.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/eff027c43288d15307676b1bd4736ab11f34c407
Modified Files
--------------
src/backend/catalog/heap.c | 10 +++-------
src/backend/catalog/index.c | 6 ++++++
src/interfaces/ecpg/test/expected/sql-fetch.stderr | 6 +++---
3 files changed, 12 insertions(+), 10 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2011-02-15 21:05:59 | Re: pgsql: Adjust pg_upgrade error message, array freeing, and add error ch |
Previous Message | Bruce Momjian | 2011-02-15 20:00:49 | pgsql: Adjust pg_upgrade error message, array freeing, and add error ch |