pgsql: Add missing handling of PlannedStmt.transientPlan in

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:22:15
Message-ID: 20100818152215.505297541DF@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.

Tags:
----
REL8_3_STABLE

Modified Files:
--------------
pgsql/src/backend/nodes:
copyfuncs.c (r1.388.2.1 -> r1.388.2.2)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/copyfuncs.c?r1=1.388.2.1&r2=1.388.2.2)
outfuncs.c (r1.322.2.2 -> r1.322.2.3)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c?r1=1.322.2.2&r2=1.322.2.3)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2010-08-18 18:35:22 pgsql: Fix failure of "ALTER TABLE t ADD COLUMN c serial" when done by
Previous Message Tom Lane 2010-08-18 15:22:09 pgsql: Add missing handling of PlannedStmt.transientPlan in