| 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: | [email protected] | 
| Views: | Whole Thread | Raw Message | 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)
| 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 |