Skip unnecessary plan-copying now that plan trees are read-only in the

executor.

diff --git a/src/backend/executor/spi.c b/src/backend/executor/spi.c
index f39c481db9310940cdad1b3a24abc42d960693d7..dba491e3d030b3c566fdd2f559a188639f7f18f7 100644 (file)
--- a/src/backend/executor/spi.c
+++ b/src/backend/executor/spi.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/executor/spi.c,v 1.81 2002/12/15 21:01:34 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/executor/spi.c,v 1.82 2002/12/17 15:51:59 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -219,9 +219,6 @@ SPI_execp(void *plan, Datum *Values, char *Nulls, int tcount)
        if (res < 0)
                return res;
 
-       /* copy plan to current (executor) context */
-       plan = (void *) _SPI_copy_plan(plan, _SPI_CPLAN_CURCXT);
-
        res = _SPI_execute_plan((_SPI_plan *) plan, Values, Nulls, tcount);
 
        _SPI_end_call(true);
@@ -1480,7 +1477,7 @@ _SPI_copy_plan(_SPI_plan *plan, int location)
                parentcxt = _SPI_current->procCxt;
        else if (location == _SPI_CPLAN_TOPCXT)
                parentcxt = TopMemoryContext;
-       else
+       else                                            /* (this case not currently used) */
                parentcxt = CurrentMemoryContext;
 
        /*