From ee33b95d9c2ecec170bc517783d7268a4bd0c793 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 9 Sep 2008 18:58:09 +0000 Subject: Improve the plan cache invalidation mechanism to make it invalidate plans when user-defined functions used in a plan are modified. Also invalidate plans when schemas, operators, or operator classes are modified; but for these cases we just invalidate everything rather than tracking exact dependencies, since these types of objects seldom change in a production database. Tom Lane; loosely based on a patch by Martin Pihlak. --- src/backend/nodes/outfuncs.c | 28 ++++++++++++++++++++++------ 1 file changed, 22 insertions(+), 6 deletions(-) (limited to 'src/backend/nodes/outfuncs.c') diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 15ced48445..0b74b3a063 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.338 2008/09/01 20:42:44 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.339 2008/09/09 18:58:08 tgl Exp $ * * NOTES * Every node type that can appear in stored rules' parsetrees *must* @@ -255,6 +255,7 @@ _outPlannedStmt(StringInfo str, PlannedStmt *node) WRITE_NODE_FIELD(returningLists); WRITE_NODE_FIELD(rowMarks); WRITE_NODE_FIELD(relationOids); + WRITE_NODE_FIELD(invalItems); WRITE_INT_FIELD(nParamExec); } @@ -593,6 +594,14 @@ _outUnique(StringInfo str, Unique *node) appendStringInfo(str, " %u", node->uniqOperators[i]); } +static void +_outHash(StringInfo str, Hash *node) +{ + WRITE_NODE_TYPE("HASH"); + + _outPlanInfo(str, (Plan *) node); +} + static void _outSetOp(StringInfo str, SetOp *node) { @@ -631,11 +640,14 @@ _outLimit(StringInfo str, Limit *node) } static void -_outHash(StringInfo str, Hash *node) +_outPlanInvalItem(StringInfo str, PlanInvalItem *node) { - WRITE_NODE_TYPE("HASH"); + WRITE_NODE_TYPE("PLANINVALITEM"); - _outPlanInfo(str, (Plan *) node); + WRITE_INT_FIELD(cacheId); + appendStringInfo(str, " :tupleId (%u,%u)", + ItemPointerGetBlockNumber(&node->tupleId), + ItemPointerGetOffsetNumber(&node->tupleId)); } /***************************************************************************** @@ -1354,6 +1366,7 @@ _outPlannerGlobal(StringInfo str, PlannerGlobal *node) WRITE_BITMAPSET_FIELD(rewindPlanIDs); WRITE_NODE_FIELD(finalrtable); WRITE_NODE_FIELD(relationOids); + WRITE_NODE_FIELD(invalItems); } static void @@ -2206,14 +2219,17 @@ _outNode(StringInfo str, void *obj) case T_Unique: _outUnique(str, obj); break; + case T_Hash: + _outHash(str, obj); + break; case T_SetOp: _outSetOp(str, obj); break; case T_Limit: _outLimit(str, obj); break; - case T_Hash: - _outHash(str, obj); + case T_PlanInvalItem: + _outPlanInvalItem(str, obj); break; case T_Alias: _outAlias(str, obj); -- cgit v1.2.1