From: | tgl(at)postgresql(dot)org (Tom Lane) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Add missing handling of PlannedStmt.transientPlan in |
Date: | 2010-08-18 15:21:54 |
Message-ID: | 20100818152154.69A857541D7@cvs.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Log Message:
-----------
Add missing handling of PlannedStmt.transientPlan in copyfuncs/outfuncs.
_outPlannedStmt is only debug support, so the omission there was not very
serious, but the omission in _copyPlannedStmt is a real bug. The consequence
would be that a copied plan tree would never be marked as a transient plan,
so that we would forget we ought to replan it after some not-yet-ready index
becomes ready for use. This might explain some past complaints about indexes
created with CREATE INDEX CONCURRENTLY not being used right away. Problem
spotted by Yeb Havinga.
Back-patch to 8.3, where the field was added.
Modified Files:
--------------
pgsql/src/backend/nodes:
copyfuncs.c (r1.467 -> r1.468)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/copyfuncs.c?r1=1.467&r2=1.468)
outfuncs.c (r1.388 -> r1.389)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c?r1=1.388&r2=1.389)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2010-08-18 15:22:00 | pgsql: Add missing handling of PlannedStmt.transientPlan in |
Previous Message | Heikki Linnakangas | 2010-08-18 15:03:34 | Re: pgsql: Coerce 'unknown' type parameters to the right type in the |