]> granicus.if.org Git - postgresql/blobdiff - src/pl/plperl/plperl.c
Fix typo in comment.
[postgresql] / src / pl / plperl / plperl.c
index 7533e5784361a404cf1d62e7e39303663204e108..da1b8780d3f8d749264a6c4dfce63c6ecc9ef3bb 100644 (file)
@@ -4,7 +4,7 @@
  * IDENTIFICATION
  *
  *       This software is copyrighted by Mark Hollomon
- *      but is shameless cribbed from pltcl.c by Jan Weick.
+ *       but is shamelessly cribbed from pltcl.c by Jan Wieck.
  *
  *       The author hereby grants permission  to  use,  copy,  modify,
  *       distribute,  and      license this software and its documentation
  *       ENHANCEMENTS, OR MODIFICATIONS.
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/pl/plperl/plperl.c,v 1.31 2002/06/15 19:54:24 momjian Exp $
+ *       $PostgreSQL: pgsql/src/pl/plperl/plperl.c,v 1.103 2006/02/28 23:38:13 neilc Exp $
  *
  **********************************************************************/
 
 #include "postgres.h"
+/* Defined by Perl */
+#undef _
 
 /* system stuff */
-#include <stdio.h>
-#include <stdlib.h>
-#include <stdarg.h>
-#include <unistd.h>
+#include <ctype.h>
 #include <fcntl.h>
-#include <string.h>
-#include <setjmp.h>
+#include <unistd.h>
+#include <locale.h>
 
 /* postgreSQL stuff */
-#include "executor/spi.h"
 #include "commands/trigger.h"
-#include "utils/elog.h"
-#include "fmgr.h"
-#include "access/heapam.h"
+#include "executor/spi.h"
+#include "funcapi.h"
+#include "utils/lsyscache.h"
+#include "utils/memutils.h"
+#include "utils/typcache.h"
+#include "miscadmin.h"
+#include "mb/pg_wchar.h"
 
-#include "tcop/tcopprot.h"
-#include "utils/syscache.h"
-#include "catalog/pg_language.h"
-#include "catalog/pg_proc.h"
-#include "catalog/pg_type.h"
+/* define this before the perl headers get a chance to mangle DLLIMPORT */
+extern DLLIMPORT bool check_function_bodies;
 
 /* perl stuff */
-#include "EXTERN.h"
-#include "perl.h"
-#include "XSUB.h"
-#include "ppport.h"
-
-/* just in case these symbols aren't provided */
-#ifndef pTHX_
-#define pTHX_
-#define pTHX void
-#endif
-
+#include "plperl.h"
 
 /**********************************************************************
  * The information we cache about loaded procedures
@@ -82,26 +71,48 @@ typedef struct plperl_proc_desc
        char       *proname;
        TransactionId fn_xmin;
        CommandId       fn_cmin;
+       bool            fn_readonly;
        bool            lanpltrusted;
-       FmgrInfo        result_in_func;
-       Oid                     result_in_elem;
+       bool            fn_retistuple;  /* true, if function returns tuple */
+       bool            fn_retisset;    /* true, if function returns set */
+       bool            fn_retisarray;  /* true if function returns array */
+       Oid                     result_oid;             /* Oid of result type */
+       FmgrInfo        result_in_func; /* I/O function and arg for result type */
+       Oid                     result_typioparam;
        int                     nargs;
        FmgrInfo        arg_out_func[FUNC_MAX_ARGS];
-       Oid                     arg_out_elem[FUNC_MAX_ARGS];
-       int                     arg_is_rel[FUNC_MAX_ARGS];
+       bool            arg_is_rowtype[FUNC_MAX_ARGS];
        SV                 *reference;
-}      plperl_proc_desc;
+} plperl_proc_desc;
+
+/*
+ * The information we cache for the duration of a single call to a
+ * function.
+ */
+typedef struct plperl_call_data
+{
+       plperl_proc_desc *prodesc;
+       FunctionCallInfo  fcinfo;
+       Tuplestorestate  *tuple_store;
+       TupleDesc                 ret_tdesc;
+       AttInMetadata    *attinmeta;
+       MemoryContext     tmp_cxt;
+} plperl_call_data;
 
 
 /**********************************************************************
  * Global data
  **********************************************************************/
-static int     plperl_firstcall = 1;
-static int     plperl_call_level = 0;
-static int     plperl_restart_in_progress = 0;
+static bool plperl_firstcall = true;
+static bool plperl_safe_init_done = false;
 static PerlInterpreter *plperl_interp = NULL;
 static HV  *plperl_proc_hash = NULL;
 
+static bool plperl_use_strict = false;
+
+/* this is saved and restored by plperl_call_handler */
+static plperl_call_data *current_call_data = NULL;
+
 /**********************************************************************
  * Forward declarations
  **********************************************************************/
@@ -109,14 +120,17 @@ static void plperl_init_all(void);
 static void plperl_init_interp(void);
 
 Datum          plperl_call_handler(PG_FUNCTION_ARGS);
+Datum          plperl_validator(PG_FUNCTION_ARGS);
+void           plperl_init(void);
 
 static Datum plperl_func_handler(PG_FUNCTION_ARGS);
 
+static Datum plperl_trigger_handler(PG_FUNCTION_ARGS);
 static plperl_proc_desc *compile_plperl_function(Oid fn_oid, bool is_trigger);
 
-static SV  *plperl_build_tuple_argument(HeapTuple tuple, TupleDesc tupdesc);
+static SV  *plperl_hash_from_tuple(HeapTuple tuple, TupleDesc tupdesc);
 static void plperl_init_shared_libs(pTHX);
-
+static HV  *plperl_spi_execute_fetch_result(SPITupleTable *, int, int);
 
 /*
  * This routine is a crock, and so is everyplace that calls it.  The problem
@@ -135,177 +149,674 @@ perm_fmgr_info(Oid functionId, FmgrInfo *finfo)
        fmgr_info_cxt(functionId, finfo, TopMemoryContext);
 }
 
-/**********************************************************************
- * plperl_init_all()           - Initialize all
- **********************************************************************/
-static void
-plperl_init_all(void)
-{
 
-       /************************************************************
-        * Do initialization only once
-        ************************************************************/
+/* Perform initialization during postmaster startup. */
+
+void
+plperl_init(void)
+{
        if (!plperl_firstcall)
                return;
 
+       DefineCustomBoolVariable(
+                                                        "plperl.use_strict",
+         "If true, will compile trusted and untrusted perl code in strict mode",
+                                                        NULL,
+                                                        &plperl_use_strict,
+                                                        PGC_USERSET,
+                                                        NULL, NULL);
 
-       /************************************************************
-        * Destroy the existing Perl interpreter
-        ************************************************************/
-       if (plperl_interp != NULL)
-       {
-               perl_destruct(plperl_interp);
-               perl_free(plperl_interp);
-               plperl_interp = NULL;
-       }
-
-       /************************************************************
-        * Free the proc hash table
-        ************************************************************/
-       if (plperl_proc_hash != NULL)
-       {
-               hv_undef(plperl_proc_hash);
-               SvREFCNT_dec((SV *) plperl_proc_hash);
-               plperl_proc_hash = NULL;
-       }
+       EmitWarningsOnPlaceholders("plperl");
 
-       /************************************************************
-        * Now recreate a new Perl interpreter
-        ************************************************************/
        plperl_init_interp();
+       plperl_firstcall = false;
+}
+
 
-       plperl_firstcall = 0;
+/* Perform initialization during backend startup. */
+
+static void
+plperl_init_all(void)
+{
+       if (plperl_firstcall)
+               plperl_init();
+
+       /* We don't need to do anything yet when a new backend starts. */
 }
 
+/* Each of these macros must represent a single string literal */
+
+#define PERLBOOT \
+       "SPI::bootstrap(); use vars qw(%_SHARED);" \
+       "sub ::plperl_warn { my $msg = shift; " \
+       "       $msg =~ s/\\(eval \\d+\\) //g; &elog(&NOTICE, $msg); } " \
+       "$SIG{__WARN__} = \\&::plperl_warn; " \
+       "sub ::plperl_die { my $msg = shift; " \
+       "       $msg =~ s/\\(eval \\d+\\) //g; die $msg; } " \
+       "$SIG{__DIE__} = \\&::plperl_die; " \
+       "sub ::mkunsafefunc {" \
+       "      my $ret = eval(qq[ sub { $_[0] $_[1] } ]); " \
+       "      $@ =~ s/\\(eval \\d+\\) //g if $@; return $ret; }" \
+       "use strict; " \
+       "sub ::mk_strict_unsafefunc {" \
+       "      my $ret = eval(qq[ sub { use strict; $_[0] $_[1] } ]); " \
+       "      $@ =~ s/\\(eval \\d+\\) //g if $@; return $ret; } " \
+       "sub ::_plperl_to_pg_array {" \
+       "  my $arg = shift; ref $arg eq 'ARRAY' || return $arg; " \
+       "  my $res = ''; my $first = 1; " \
+       "  foreach my $elem (@$arg) " \
+       "  { " \
+       "    $res .= ', ' unless $first; $first = undef; " \
+       "    if (ref $elem) " \
+       "    { " \
+       "      $res .= _plperl_to_pg_array($elem); " \
+       "    } " \
+       "    elsif (defined($elem)) " \
+       "    { " \
+       "      my $str = qq($elem); " \
+       "      $str =~ s/([\"\\\\])/\\\\$1/g; " \
+       "      $res .= qq(\"$str\"); " \
+       "    } " \
+       "    else " \
+       "    { "\
+       "      $res .= 'NULL' ; " \
+       "    } "\
+       "  } " \
+       "  return qq({$res}); " \
+       "} "
+
+#define SAFE_MODULE \
+       "require Safe; $Safe::VERSION"
+
+#define SAFE_OK \
+       "use vars qw($PLContainer); $PLContainer = new Safe('PLPerl');" \
+       "$PLContainer->permit_only(':default');" \
+       "$PLContainer->permit(qw[:base_math !:base_io sort time]);" \
+       "$PLContainer->share(qw[&elog &spi_exec_query &return_next " \
+       "&spi_query &spi_fetchrow " \
+       "&_plperl_to_pg_array " \
+       "&DEBUG &LOG &INFO &NOTICE &WARNING &ERROR %_SHARED ]);" \
+       "sub ::mksafefunc {" \
+       "      my $ret = $PLContainer->reval(qq[sub { $_[0] $_[1] }]); " \
+       "      $@ =~ s/\\(eval \\d+\\) //g if $@; return $ret; }" \
+       "$PLContainer->permit('require'); $PLContainer->reval('use strict;');" \
+       "$PLContainer->deny('require');" \
+       "sub ::mk_strict_safefunc {" \
+       "      my $ret = $PLContainer->reval(qq[sub { BEGIN { strict->import(); } $_[0] $_[1] }]); " \
+       "      $@ =~ s/\\(eval \\d+\\) //g if $@; return $ret; }"
+
+#define SAFE_BAD \
+       "use vars qw($PLContainer); $PLContainer = new Safe('PLPerl');" \
+       "$PLContainer->permit_only(':default');" \
+       "$PLContainer->share(qw[&elog &ERROR ]);" \
+       "sub ::mksafefunc { return $PLContainer->reval(qq[sub { " \
+       "      elog(ERROR,'trusted Perl functions disabled - " \
+       "      please upgrade Perl Safe module to version 2.09 or later');}]); }" \
+       "sub ::mk_strict_safefunc { return $PLContainer->reval(qq[sub { " \
+       "      elog(ERROR,'trusted Perl functions disabled - " \
+       "      please upgrade Perl Safe module to version 2.09 or later');}]); }"
+
 
-/**********************************************************************
- * plperl_init_interp() - Create the Perl interpreter
- **********************************************************************/
 static void
 plperl_init_interp(void)
 {
+       static char *embedding[3] = {
+               "", "-e", PERLBOOT
+       };
 
-       char       *embedding[3] = {
-               "", "-e",
+#ifdef WIN32
 
-               /*
-                * no commas between the next 5 please. They are supposed to be
-                * one string
-                */
-               "require Safe; SPI::bootstrap();"
-               "sub ::mksafefunc { my $x = new Safe; $x->permit_only(':default');$x->permit(':base_math');"
-               "$x->share(qw[&elog &DEBUG &LOG &INFO &NOTICE &WARNING &ERROR]);"
-               " return $x->reval(qq[sub { $_[0] }]); }"
-               "sub ::mkunsafefunc {return eval(qq[ sub { $_[0] } ]); }"
-       };
+       /* 
+        * The perl library on startup does horrible things like call
+        * setlocale(LC_ALL,""). We have protected against that on most
+        * platforms by setting the environment appropriately. However, on
+        * Windows, setlocale() does not consult the environment, so we need
+        * to save the existing locale settings before perl has a chance to 
+        * mangle them and restore them after its dirty deeds are done.
+        *
+        * MSDN ref:
+        * http://msdn.microsoft.com/library/en-us/vclib/html/_crt_locale.asp
+        *
+        * It appears that we only need to do this on interpreter startup, and
+        * subsequent calls to the interpreter don't mess with the locale
+        * settings.
+        *
+        * We restore them using Perl's POSIX::setlocale() function so that
+        * Perl doesn't have a different idea of the locale from Postgres.
+        *
+        */
+
+       char *loc;
+       char *save_collate, *save_ctype, *save_monetary, *save_numeric, *save_time;
+       char buf[1024];
+
+       loc = setlocale(LC_COLLATE,NULL);
+       save_collate = loc ? pstrdup(loc) : NULL;
+       loc = setlocale(LC_CTYPE,NULL);
+       save_ctype = loc ? pstrdup(loc) : NULL;
+       loc = setlocale(LC_MONETARY,NULL);
+       save_monetary = loc ? pstrdup(loc) : NULL;
+       loc = setlocale(LC_NUMERIC,NULL);
+       save_numeric = loc ? pstrdup(loc) : NULL;
+       loc = setlocale(LC_TIME,NULL);
+       save_time = loc ? pstrdup(loc) : NULL;
+
+#endif
 
        plperl_interp = perl_alloc();
        if (!plperl_interp)
-               elog(ERROR, "plperl_init_interp(): could not allocate perl interpreter");
+               elog(ERROR, "could not allocate Perl interpreter");
 
        perl_construct(plperl_interp);
        perl_parse(plperl_interp, plperl_init_shared_libs, 3, embedding, NULL);
        perl_run(plperl_interp);
 
+       plperl_proc_hash = newHV();
 
+#ifdef WIN32
 
-       /************************************************************
-        * Initialize the proc and query hash tables
-        ************************************************************/
-       plperl_proc_hash = newHV();
+       eval_pv("use POSIX qw(locale_h);", TRUE); /* croak on failure */
+
+       if (save_collate != NULL)
+       {
+               snprintf(buf, sizeof(buf),"setlocale(%s,'%s');",
+                                "LC_COLLATE",save_collate);
+               eval_pv(buf,TRUE);
+               pfree(save_collate);
+       }
+       if (save_ctype != NULL)
+       {
+               snprintf(buf, sizeof(buf),"setlocale(%s,'%s');",
+                                "LC_CTYPE",save_ctype);
+               eval_pv(buf,TRUE);
+               pfree(save_ctype);
+       }
+       if (save_monetary != NULL)
+       {
+               snprintf(buf, sizeof(buf),"setlocale(%s,'%s');",
+                                "LC_MONETARY",save_monetary);
+               eval_pv(buf,TRUE);
+               pfree(save_monetary);
+       }
+       if (save_numeric != NULL)
+       {
+               snprintf(buf, sizeof(buf),"setlocale(%s,'%s');",
+                                "LC_NUMERIC",save_numeric);
+               eval_pv(buf,TRUE);
+               pfree(save_numeric);
+       }
+       if (save_time != NULL)
+       {
+               snprintf(buf, sizeof(buf),"setlocale(%s,'%s');",
+                                "LC_TIME",save_time);
+               eval_pv(buf,TRUE);
+               pfree(save_time);
+       }
+
+#endif
 
 }
 
 
+static void
+plperl_safe_init(void)
+{
+       SV                 *res;
+       double          safe_version;
 
-/**********************************************************************
- * plperl_call_handler         - This is the only visible function
- *                               of the PL interpreter. The PostgreSQL
- *                               function manager and trigger manager
- *                               call this function for execution of
- *                               perl procedures.
- **********************************************************************/
+       res = eval_pv(SAFE_MODULE, FALSE);      /* TRUE = croak if failure */
+
+       safe_version = SvNV(res);
+
+       /*
+        * We actually want to reject safe_version < 2.09, but it's risky to
+        * assume that floating-point comparisons are exact, so use a slightly
+        * smaller comparison value.
+        */
+       if (safe_version < 2.0899)
+       {
+               /* not safe, so disallow all trusted funcs */
+               eval_pv(SAFE_BAD, FALSE);
+       }
+       else
+       {
+               eval_pv(SAFE_OK, FALSE);
+       }
+
+       plperl_safe_init_done = true;
+}
+
+/*
+ * Perl likes to put a newline after its error messages; clean up such
+ */
+static char *
+strip_trailing_ws(const char *msg)
+{
+       char       *res = pstrdup(msg);
+       int                     len = strlen(res);
+
+       while (len > 0 && isspace((unsigned char) res[len - 1]))
+               res[--len] = '\0';
+       return res;
+}
+
+
+/* Build a tuple from a hash. */
+
+static HeapTuple
+plperl_build_tuple_result(HV *perlhash, AttInMetadata *attinmeta)
+{
+       TupleDesc       td = attinmeta->tupdesc;
+       char      **values;
+       SV                 *val;
+       char       *key;
+       I32                     klen;
+       HeapTuple       tup;
+
+       values = (char **) palloc0(td->natts * sizeof(char *));
+
+       hv_iterinit(perlhash);
+       while ((val = hv_iternextsv(perlhash, &key, &klen)))
+       {
+               int                     attn = SPI_fnumber(td, key);
+
+               if (attn <= 0 || td->attrs[attn - 1]->attisdropped)
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_UNDEFINED_COLUMN),
+                                        errmsg("Perl hash contains nonexistent column \"%s\"",
+                                                       key)));
+               if (SvOK(val) && SvTYPE(val) != SVt_NULL)
+                       values[attn - 1] = SvPV(val, PL_na);
+       }
+       hv_iterinit(perlhash);
+
+       tup = BuildTupleFromCStrings(attinmeta, values);
+       pfree(values);
+       return tup;
+}
+
+/*
+ * convert perl array to postgres string representation
+ */
+static SV  *
+plperl_convert_to_pg_array(SV *src)
+{
+       SV                 *rv;
+       int                     count;
+
+       dSP;
+
+       PUSHMARK(SP);
+       XPUSHs(src);
+       PUTBACK;
+
+       count = call_pv("::_plperl_to_pg_array", G_SCALAR);
+
+       SPAGAIN;
+
+       if (count != 1)
+               elog(ERROR, "unexpected _plperl_to_pg_array failure");
+
+       rv = POPs;
+
+       PUTBACK;
+
+       return rv;
+}
+
+
+/* Set up the arguments for a trigger call. */
+
+static SV  *
+plperl_trigger_build_args(FunctionCallInfo fcinfo)
+{
+       TriggerData *tdata;
+       TupleDesc       tupdesc;
+       int                     i;
+       char       *level;
+       char       *event;
+       char       *relid;
+       char       *when;
+       HV                 *hv;
+
+       hv = newHV();
+
+       tdata = (TriggerData *) fcinfo->context;
+       tupdesc = tdata->tg_relation->rd_att;
+
+       relid = DatumGetCString(
+                                                       DirectFunctionCall1(oidout,
+                                                                 ObjectIdGetDatum(tdata->tg_relation->rd_id)
+                                                                                               )
+               );
+
+       hv_store(hv, "name", 4, newSVpv(tdata->tg_trigger->tgname, 0), 0);
+       hv_store(hv, "relid", 5, newSVpv(relid, 0), 0);
+
+       if (TRIGGER_FIRED_BY_INSERT(tdata->tg_event))
+       {
+               event = "INSERT";
+               if (TRIGGER_FIRED_FOR_ROW(tdata->tg_event))
+                       hv_store(hv, "new", 3,
+                                        plperl_hash_from_tuple(tdata->tg_trigtuple, tupdesc),
+                                        0);
+       }
+       else if (TRIGGER_FIRED_BY_DELETE(tdata->tg_event))
+       {
+               event = "DELETE";
+               if (TRIGGER_FIRED_FOR_ROW(tdata->tg_event))
+                       hv_store(hv, "old", 3,
+                                        plperl_hash_from_tuple(tdata->tg_trigtuple, tupdesc),
+                                        0);
+       }
+       else if (TRIGGER_FIRED_BY_UPDATE(tdata->tg_event))
+       {
+               event = "UPDATE";
+               if (TRIGGER_FIRED_FOR_ROW(tdata->tg_event))
+               {
+                       hv_store(hv, "old", 3,
+                                        plperl_hash_from_tuple(tdata->tg_trigtuple, tupdesc),
+                                        0);
+                       hv_store(hv, "new", 3,
+                                        plperl_hash_from_tuple(tdata->tg_newtuple, tupdesc),
+                                        0);
+               }
+       }
+       else
+               event = "UNKNOWN";
+
+       hv_store(hv, "event", 5, newSVpv(event, 0), 0);
+       hv_store(hv, "argc", 4, newSViv(tdata->tg_trigger->tgnargs), 0);
+
+       if (tdata->tg_trigger->tgnargs > 0)
+       {
+               AV                 *av = newAV();
+
+               for (i = 0; i < tdata->tg_trigger->tgnargs; i++)
+                       av_push(av, newSVpv(tdata->tg_trigger->tgargs[i], 0));
+               hv_store(hv, "args", 4, newRV_noinc((SV *) av), 0);
+       }
+
+       hv_store(hv, "relname", 7,
+                        newSVpv(SPI_getrelname(tdata->tg_relation), 0), 0);
+
+       if (TRIGGER_FIRED_BEFORE(tdata->tg_event))
+               when = "BEFORE";
+       else if (TRIGGER_FIRED_AFTER(tdata->tg_event))
+               when = "AFTER";
+       else
+               when = "UNKNOWN";
+       hv_store(hv, "when", 4, newSVpv(when, 0), 0);
+
+       if (TRIGGER_FIRED_FOR_ROW(tdata->tg_event))
+               level = "ROW";
+       else if (TRIGGER_FIRED_FOR_STATEMENT(tdata->tg_event))
+               level = "STATEMENT";
+       else
+               level = "UNKNOWN";
+       hv_store(hv, "level", 5, newSVpv(level, 0), 0);
+
+       return newRV_noinc((SV *) hv);
+}
+
+
+/* Set up the new tuple returned from a trigger. */
+
+static HeapTuple
+plperl_modify_tuple(HV *hvTD, TriggerData *tdata, HeapTuple otup)
+{
+       SV                **svp;
+       HV                 *hvNew;
+       HeapTuple       rtup;
+       SV                 *val;
+       char       *key;
+       I32                     klen;
+       int                     slotsused;
+       int                *modattrs;
+       Datum      *modvalues;
+       char       *modnulls;
+
+       TupleDesc       tupdesc;
+
+       tupdesc = tdata->tg_relation->rd_att;
+
+       svp = hv_fetch(hvTD, "new", 3, FALSE);
+       if (!svp)
+               ereport(ERROR,
+                               (errcode(ERRCODE_UNDEFINED_COLUMN),
+                                errmsg("$_TD->{new} does not exist")));
+       if (!SvOK(*svp) || SvTYPE(*svp) != SVt_RV || SvTYPE(SvRV(*svp)) != SVt_PVHV)
+               ereport(ERROR,
+                               (errcode(ERRCODE_DATATYPE_MISMATCH),
+                                errmsg("$_TD->{new} is not a hash reference")));
+       hvNew = (HV *) SvRV(*svp);
+
+       modattrs = palloc(tupdesc->natts * sizeof(int));
+       modvalues = palloc(tupdesc->natts * sizeof(Datum));
+       modnulls = palloc(tupdesc->natts * sizeof(char));
+       slotsused = 0;
+
+       hv_iterinit(hvNew);
+       while ((val = hv_iternextsv(hvNew, &key, &klen)))
+       {
+               int                     attn = SPI_fnumber(tupdesc, key);
+
+               if (attn <= 0 || tupdesc->attrs[attn - 1]->attisdropped)
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_UNDEFINED_COLUMN),
+                                        errmsg("Perl hash contains nonexistent column \"%s\"",
+                                                       key)));
+               if (SvOK(val) && SvTYPE(val) != SVt_NULL)
+               {
+                       Oid                     typinput;
+                       Oid                     typioparam;
+                       FmgrInfo        finfo;
+
+                       /* XXX would be better to cache these lookups */
+                       getTypeInputInfo(tupdesc->attrs[attn - 1]->atttypid,
+                                                        &typinput, &typioparam);
+                       fmgr_info(typinput, &finfo);
+                       modvalues[slotsused] = FunctionCall3(&finfo,
+                                                                                  CStringGetDatum(SvPV(val, PL_na)),
+                                                                                                ObjectIdGetDatum(typioparam),
+                                                Int32GetDatum(tupdesc->attrs[attn - 1]->atttypmod));
+                       modnulls[slotsused] = ' ';
+               }
+               else
+               {
+                       modvalues[slotsused] = (Datum) 0;
+                       modnulls[slotsused] = 'n';
+               }
+               modattrs[slotsused] = attn;
+               slotsused++;
+       }
+       hv_iterinit(hvNew);
+
+       rtup = SPI_modifytuple(tdata->tg_relation, otup, slotsused,
+                                                  modattrs, modvalues, modnulls);
+
+       pfree(modattrs);
+       pfree(modvalues);
+       pfree(modnulls);
+
+       if (rtup == NULL)
+               elog(ERROR, "SPI_modifytuple failed: %s",
+                        SPI_result_code_string(SPI_result));
+
+       return rtup;
+}
+
+
+/*
+ * This is the only externally-visible part of the plperl call interface.
+ * The Postgres function and trigger managers call it to execute a
+ * perl function.
+ */
 PG_FUNCTION_INFO_V1(plperl_call_handler);
 
-/* keep non-static */
 Datum
 plperl_call_handler(PG_FUNCTION_ARGS)
 {
        Datum           retval;
+       plperl_call_data *save_call_data;
 
-       /************************************************************
-        * Initialize interpreters on first call
-        ************************************************************/
-       if (plperl_firstcall)
-               plperl_init_all();
+       plperl_init_all();
 
-       /************************************************************
-        * Connect to SPI manager
-        ************************************************************/
-       if (SPI_connect() != SPI_OK_CONNECT)
-               elog(ERROR, "plperl: cannot connect to SPI manager");
-       /************************************************************
-        * Keep track about the nesting of Perl-SPI-Perl-... calls
-        ************************************************************/
-       plperl_call_level++;
+       save_call_data = current_call_data;
+       PG_TRY();
+       {
+               if (CALLED_AS_TRIGGER(fcinfo))
+                       retval = PointerGetDatum(plperl_trigger_handler(fcinfo));
+               else
+                       retval = plperl_func_handler(fcinfo);
+       }
+       PG_CATCH();
+       {
+               current_call_data = save_call_data;
+               PG_RE_THROW();
+       }
+       PG_END_TRY();
 
-       /************************************************************
-        * Determine if called as function or trigger and
-        * call appropriate subhandler
-        ************************************************************/
-       if (CALLED_AS_TRIGGER(fcinfo))
+       current_call_data = save_call_data;
+       return retval;
+}
+
+/*
+ * This is the other externally visible function - it is called when CREATE
+ * FUNCTION is issued to validate the function being created/replaced.
+ */
+PG_FUNCTION_INFO_V1(plperl_validator);
+
+Datum
+plperl_validator(PG_FUNCTION_ARGS)
+{
+       Oid                     funcoid = PG_GETARG_OID(0);
+       HeapTuple       tuple;
+       Form_pg_proc proc;
+       char            functyptype;
+       int                     numargs;
+       Oid                *argtypes;
+       char      **argnames;
+       char       *argmodes;
+       bool            istrigger = false;
+       int                     i;
+
+       /* Get the new function's pg_proc entry */
+       tuple = SearchSysCache(PROCOID,
+                                                  ObjectIdGetDatum(funcoid),
+                                                  0, 0, 0);
+       if (!HeapTupleIsValid(tuple))
+               elog(ERROR, "cache lookup failed for function %u", funcoid);
+       proc = (Form_pg_proc) GETSTRUCT(tuple);
+
+       functyptype = get_typtype(proc->prorettype);
+
+       /* Disallow pseudotype result */
+       /* except for TRIGGER, RECORD, or VOID */
+       if (functyptype == 'p')
        {
-               elog(ERROR, "plperl: can't use perl in triggers yet.");
+               /* we assume OPAQUE with no arguments means a trigger */
+               if (proc->prorettype == TRIGGEROID ||
+                       (proc->prorettype == OPAQUEOID && proc->pronargs == 0))
+                       istrigger = true;
+               else if (proc->prorettype != RECORDOID &&
+                                proc->prorettype != VOIDOID)
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                        errmsg("plperl functions cannot return type %s",
+                                                       format_type_be(proc->prorettype))));
+       }
 
-               /*
-                * retval = PointerGetDatum(plperl_trigger_handler(fcinfo));
-                */
-               /* make the compiler happy */
-               retval = (Datum) 0;
+       /* Disallow pseudotypes in arguments (either IN or OUT) */
+       numargs = get_func_arg_info(tuple,
+                                                               &argtypes, &argnames, &argmodes);
+       for (i = 0; i < numargs; i++)
+       {
+               if (get_typtype(argtypes[i]) == 'p')
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                        errmsg("plperl functions cannot take type %s",
+                                                       format_type_be(argtypes[i]))));
        }
-       else
-               retval = plperl_func_handler(fcinfo);
 
-       plperl_call_level--;
+       ReleaseSysCache(tuple);
 
-       return retval;
+       /* Postpone body checks if !check_function_bodies */
+       if (check_function_bodies)
+       {
+               plperl_proc_desc *prodesc;
+
+               plperl_init_all();
+
+               prodesc = compile_plperl_function(funcoid, istrigger);
+       }
+
+       /* the result of a validator is ignored */
+       PG_RETURN_VOID();
 }
 
 
-/**********************************************************************
- * plperl_create_sub()         - calls the perl interpreter to
- *             create the anonymous subroutine whose text is in the SV.
- *             Returns the SV containing the RV to the closure.
- **********************************************************************/
-static
-SV *
+/* Uses mksafefunc/mkunsafefunc to create an anonymous sub whose text is
+ * supplied in s, and returns a reference to the closure. */
+
+static SV  *
 plperl_create_sub(char *s, bool trusted)
 {
        dSP;
-
-       SV                 *subref = NULL;
+       SV                 *subref;
        int                     count;
+       char       *compile_sub;
+
+       if (trusted && !plperl_safe_init_done)
+       {
+               plperl_safe_init();
+               SPAGAIN;
+       }
 
        ENTER;
        SAVETMPS;
        PUSHMARK(SP);
+       XPUSHs(sv_2mortal(newSVpv("my $_TD=$_[0]; shift;", 0)));
        XPUSHs(sv_2mortal(newSVpv(s, 0)));
        PUTBACK;
-       count = perl_call_pv((trusted ? "mksafefunc" : "mkunsafefunc"),
-                                                G_SCALAR | G_EVAL | G_KEEPERR);
+
+       /*
+        * G_KEEPERR seems to be needed here, else we don't recognize compile
+        * errors properly.  Perhaps it's because there's another level of eval
+        * inside mksafefunc?
+        */
+
+       if (trusted && plperl_use_strict)
+               compile_sub = "::mk_strict_safefunc";
+       else if (plperl_use_strict)
+               compile_sub = "::mk_strict_unsafefunc";
+       else if (trusted)
+               compile_sub = "::mksafefunc";
+       else
+               compile_sub = "::mkunsafefunc";
+
+       count = perl_call_pv(compile_sub, G_SCALAR | G_EVAL | G_KEEPERR);
        SPAGAIN;
 
-       if (SvTRUE(ERRSV))
+       if (count != 1)
        {
-               POPs;
                PUTBACK;
                FREETMPS;
                LEAVE;
-               elog(ERROR, "creation of function failed: %s", SvPV(ERRSV, PL_na));
+               elog(ERROR, "didn't get a return item from mksafefunc");
        }
 
-       if (count != 1)
-               elog(ERROR, "creation of function failed - no return from mksafefunc");
+       if (SvTRUE(ERRSV))
+       {
+               (void) POPs;
+               PUTBACK;
+               FREETMPS;
+               LEAVE;
+               ereport(ERROR,
+                               (errcode(ERRCODE_SYNTAX_ERROR),
+                                errmsg("creation of Perl function failed: %s",
+                                               strip_trailing_ws(SvPV(ERRSV, PL_na)))));
+       }
 
        /*
         * need to make a deep copy of the return. it comes off the stack as a
@@ -313,7 +824,7 @@ plperl_create_sub(char *s, bool trusted)
         */
        subref = newSVsv(POPs);
 
-       if (!SvROK(subref))
+       if (!SvROK(subref) || SvTYPE(SvRV(subref)) != SVt_PVCV)
        {
                PUTBACK;
                FREETMPS;
@@ -323,15 +834,17 @@ plperl_create_sub(char *s, bool trusted)
                 * subref is our responsibility because it is not mortal
                 */
                SvREFCNT_dec(subref);
-               elog(ERROR, "plperl_create_sub: didn't get a code ref");
+               elog(ERROR, "didn't get a code ref");
        }
 
        PUTBACK;
        FREETMPS;
        LEAVE;
+
        return subref;
 }
 
+
 /**********************************************************************
  * plperl_init_shared_libs()           -
  *
@@ -341,8 +854,8 @@ plperl_create_sub(char *s, bool trusted)
  *
  **********************************************************************/
 
-EXTERN_C void boot_DynaLoader(pTHX_ CVcv);
-EXTERN_C void boot_SPI(pTHX_ CVcv);
+EXTERN_C void boot_DynaLoader(pTHX_ CV *cv);
+EXTERN_C void boot_SPI(pTHX_ CV *cv);
 
 static void
 plperl_init_shared_libs(pTHX)
@@ -353,163 +866,405 @@ plperl_init_shared_libs(pTHX)
        newXS("SPI::bootstrap", boot_SPI, file);
 }
 
-/**********************************************************************
- * plperl_call_perl_func()             - calls a perl function through the RV
- *                     stored in the prodesc structure. massages the input parms properly
- **********************************************************************/
-static
-SV *
-plperl_call_perl_func(plperl_proc_desc * desc, FunctionCallInfo fcinfo)
+
+static SV  *
+plperl_call_perl_func(plperl_proc_desc *desc, FunctionCallInfo fcinfo)
 {
        dSP;
-
        SV                 *retval;
        int                     i;
        int                     count;
-
+       SV                 *sv;
 
        ENTER;
        SAVETMPS;
 
-       PUSHMARK(sp);
+       PUSHMARK(SP);
+
+       XPUSHs(&PL_sv_undef);           /* no trigger data */
+
        for (i = 0; i < desc->nargs; i++)
        {
-               if (desc->arg_is_rel[i])
+               if (fcinfo->argnull[i])
+                       XPUSHs(&PL_sv_undef);
+               else if (desc->arg_is_rowtype[i])
                {
-                       TupleTableSlot *slot = (TupleTableSlot *) fcinfo->arg[i];
+                       HeapTupleHeader td;
+                       Oid                     tupType;
+                       int32           tupTypmod;
+                       TupleDesc       tupdesc;
+                       HeapTupleData tmptup;
                        SV                 *hashref;
 
-                       Assert(slot != NULL && !fcinfo->argnull[i]);
-
-                       /*
-                        * plperl_build_tuple_argument better return a mortal SV.
-                        */
-                       hashref = plperl_build_tuple_argument(slot->val,
-                                                                                         slot->ttc_tupleDescriptor);
-                       XPUSHs(hashref);
+                       td = DatumGetHeapTupleHeader(fcinfo->arg[i]);
+                       /* Extract rowtype info and find a tupdesc */
+                       tupType = HeapTupleHeaderGetTypeId(td);
+                       tupTypmod = HeapTupleHeaderGetTypMod(td);
+                       tupdesc = lookup_rowtype_tupdesc(tupType, tupTypmod);
+                       /* Build a temporary HeapTuple control structure */
+                       tmptup.t_len = HeapTupleHeaderGetDatumLength(td);
+                       tmptup.t_data = td;
+
+                       hashref = plperl_hash_from_tuple(&tmptup, tupdesc);
+                       XPUSHs(sv_2mortal(hashref));
                }
                else
                {
-                       if (fcinfo->argnull[i])
-                               XPUSHs(&PL_sv_undef);
-                       else
+                       char       *tmp;
+
+                       tmp = DatumGetCString(FunctionCall1(&(desc->arg_out_func[i]),
+                                                                                               fcinfo->arg[i]));
+                       sv = newSVpv(tmp, 0);
+#if PERL_BCDVERSION >= 0x5006000L
+                       if (GetDatabaseEncoding() == PG_UTF8)
+                               SvUTF8_on(sv);
+#endif
+                       XPUSHs(sv_2mortal(sv));
+                       pfree(tmp);
+               }
+       }
+       PUTBACK;
+
+       /* Do NOT use G_KEEPERR here */
+       count = perl_call_sv(desc->reference, G_SCALAR | G_EVAL);
+
+       SPAGAIN;
+
+       if (count != 1)
+       {
+               PUTBACK;
+               FREETMPS;
+               LEAVE;
+               elog(ERROR, "didn't get a return item from function");
+       }
+
+       if (SvTRUE(ERRSV))
+       {
+               (void) POPs;
+               PUTBACK;
+               FREETMPS;
+               LEAVE;
+               /* XXX need to find a way to assign an errcode here */
+               ereport(ERROR,
+                               (errmsg("error from Perl function: %s",
+                                               strip_trailing_ws(SvPV(ERRSV, PL_na)))));
+       }
+
+       retval = newSVsv(POPs);
+
+       PUTBACK;
+       FREETMPS;
+       LEAVE;
+
+       return retval;
+}
+
+
+static SV  *
+plperl_call_perl_trigger_func(plperl_proc_desc *desc, FunctionCallInfo fcinfo,
+                                                         SV *td)
+{
+       dSP;
+       SV                 *retval;
+       Trigger    *tg_trigger;
+       int                     i;
+       int                     count;
+
+       ENTER;
+       SAVETMPS;
+
+       PUSHMARK(sp);
+
+       XPUSHs(td);
+
+       tg_trigger = ((TriggerData *) fcinfo->context)->tg_trigger;
+       for (i = 0; i < tg_trigger->tgnargs; i++)
+               XPUSHs(sv_2mortal(newSVpv(tg_trigger->tgargs[i], 0)));
+       PUTBACK;
+
+       /* Do NOT use G_KEEPERR here */
+       count = perl_call_sv(desc->reference, G_SCALAR | G_EVAL);
+
+       SPAGAIN;
+
+       if (count != 1)
+       {
+               PUTBACK;
+               FREETMPS;
+               LEAVE;
+               elog(ERROR, "didn't get a return item from trigger function");
+       }
+
+       if (SvTRUE(ERRSV))
+       {
+               (void) POPs;
+               PUTBACK;
+               FREETMPS;
+               LEAVE;
+               /* XXX need to find a way to assign an errcode here */
+               ereport(ERROR,
+                               (errmsg("error from Perl trigger function: %s",
+                                               strip_trailing_ws(SvPV(ERRSV, PL_na)))));
+       }
+
+       retval = newSVsv(POPs);
+
+       PUTBACK;
+       FREETMPS;
+       LEAVE;
+
+       return retval;
+}
+
+
+static Datum
+plperl_func_handler(PG_FUNCTION_ARGS)
+{
+       plperl_proc_desc *prodesc;
+       SV                 *perlret;
+       Datum           retval;
+       ReturnSetInfo *rsi;
+       SV                 *array_ret = NULL;
+
+       /*
+        * Create the call_data beforing connecting to SPI, so that it is
+        * not allocated in the SPI memory context
+        */
+       current_call_data = (plperl_call_data *) palloc0(sizeof(plperl_call_data));
+       current_call_data->fcinfo = fcinfo;
+
+       if (SPI_connect() != SPI_OK_CONNECT)
+               elog(ERROR, "could not connect to SPI manager");
+
+       prodesc = compile_plperl_function(fcinfo->flinfo->fn_oid, false);
+       current_call_data->prodesc = prodesc;
+
+       rsi = (ReturnSetInfo *) fcinfo->resultinfo;
+
+       if (prodesc->fn_retisset)
+       {
+               /* Check context before allowing the call to go through */
+               if (!rsi || !IsA(rsi, ReturnSetInfo) ||
+                       (rsi->allowedModes & SFRM_Materialize) == 0 ||
+                       rsi->expectedDesc == NULL)
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                        errmsg("set-valued function called in context that "
+                                                       "cannot accept a set")));
+       }
+
+       perlret = plperl_call_perl_func(prodesc, fcinfo);
+
+       /************************************************************
+        * Disconnect from SPI manager and then create the return
+        * values datum (if the input function does a palloc for it
+        * this must not be allocated in the SPI memory context
+        * because SPI_finish would free it).
+        ************************************************************/
+       if (SPI_finish() != SPI_OK_FINISH)
+               elog(ERROR, "SPI_finish() failed");
+
+       if (prodesc->fn_retisset)
+       {
+               /*
+                * If the Perl function returned an arrayref, we pretend that it
+                * called return_next() for each element of the array, to handle old
+                * SRFs that didn't know about return_next(). Any other sort of return
+                * value is an error.
+                */
+               if (SvTYPE(perlret) == SVt_RV &&
+                       SvTYPE(SvRV(perlret)) == SVt_PVAV)
+               {
+                       int                     i = 0;
+                       SV                **svp = 0;
+                       AV                 *rav = (AV *) SvRV(perlret);
+
+                       while ((svp = av_fetch(rav, i, FALSE)) != NULL)
                        {
-                               char       *tmp;
-
-                               tmp = DatumGetCString(FunctionCall3(&(desc->arg_out_func[i]),
-                                                                                                       fcinfo->arg[i],
-                                                                ObjectIdGetDatum(desc->arg_out_elem[i]),
-                                                                                                       Int32GetDatum(-1)));
-                               XPUSHs(sv_2mortal(newSVpv(tmp, 0)));
-                               pfree(tmp);
+                               plperl_return_next(*svp);
+                               i++;
                        }
                }
+               else if (SvTYPE(perlret) != SVt_NULL)
+               {
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_DATATYPE_MISMATCH),
+                                        errmsg("set-returning Perl function must return "
+                                                       "reference to array or use return_next")));
+               }
+
+               rsi->returnMode = SFRM_Materialize;
+               if (current_call_data->tuple_store)
+               {
+                       rsi->setResult = current_call_data->tuple_store;
+                       rsi->setDesc = current_call_data->ret_tdesc;
+               }
+               retval = (Datum) 0;
        }
-       PUTBACK;
-       count = perl_call_sv(desc->reference, G_SCALAR | G_EVAL | G_KEEPERR);
+       else if (SvTYPE(perlret) == SVt_NULL)
+       {
+               /* Return NULL if Perl code returned undef */
+               if (rsi && IsA(rsi, ReturnSetInfo))
+                       rsi->isDone = ExprEndResult;
+               fcinfo->isnull = true;
+               retval = (Datum) 0;
+       }
+       else if (prodesc->fn_retistuple)
+       {
+               /* Return a perl hash converted to a Datum */
+               TupleDesc       td;
+               AttInMetadata *attinmeta;
+               HeapTuple       tup;
+
+               if (!SvOK(perlret) || SvTYPE(perlret) != SVt_RV ||
+                       SvTYPE(SvRV(perlret)) != SVt_PVHV)
+               {
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_DATATYPE_MISMATCH),
+                                        errmsg("composite-returning Perl function "
+                                                       "must return reference to hash")));
+               }
 
-       SPAGAIN;
+               /* XXX should cache the attinmeta data instead of recomputing */
+               if (get_call_result_type(fcinfo, NULL, &td) != TYPEFUNC_COMPOSITE)
+               {
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                        errmsg("function returning record called in context "
+                                                       "that cannot accept type record")));
+               }
 
-       if (count != 1)
-       {
-               PUTBACK;
-               FREETMPS;
-               LEAVE;
-               elog(ERROR, "plperl: didn't get a return item from function");
+               attinmeta = TupleDescGetAttInMetadata(td);
+               tup = plperl_build_tuple_result((HV *) SvRV(perlret), attinmeta);
+               retval = HeapTupleGetDatum(tup);
        }
-
-       if (SvTRUE(ERRSV))
+       else
        {
-               POPs;
-               PUTBACK;
-               FREETMPS;
-               LEAVE;
-               elog(ERROR, "plperl: error from function: %s", SvPV(ERRSV, PL_na));
-       }
-
-       retval = newSVsv(POPs);
+               /* Return a perl string converted to a Datum */
+               char       *val;
 
+               if (prodesc->fn_retisarray && SvROK(perlret) &&
+                       SvTYPE(SvRV(perlret)) == SVt_PVAV)
+               {
+                       array_ret = plperl_convert_to_pg_array(perlret);
+                       SvREFCNT_dec(perlret);
+                       perlret = array_ret;
+               }
 
-       PUTBACK;
-       FREETMPS;
-       LEAVE;
+               val = SvPV(perlret, PL_na);
 
-       return retval;
+               retval = FunctionCall3(&prodesc->result_in_func,
+                                                          CStringGetDatum(val),
+                                                          ObjectIdGetDatum(prodesc->result_typioparam),
+                                                          Int32GetDatum(-1));
+       }
 
+       if (array_ret == NULL)
+               SvREFCNT_dec(perlret);
 
+       current_call_data = NULL;
+       return retval;
 }
 
-/**********************************************************************
- * plperl_func_handler()               - Handler for regular function calls
- **********************************************************************/
+
 static Datum
-plperl_func_handler(PG_FUNCTION_ARGS)
+plperl_trigger_handler(PG_FUNCTION_ARGS)
 {
        plperl_proc_desc *prodesc;
        SV                 *perlret;
        Datum           retval;
-       sigjmp_buf      save_restart;
+       SV                 *svTD;
+       HV                 *hvTD;
 
-       /* Find or compile the function */
-       prodesc = compile_plperl_function(fcinfo->flinfo->fn_oid, false);
+       /*
+        * Create the call_data beforing connecting to SPI, so that it is
+        * not allocated in the SPI memory context
+        */
+       current_call_data = (plperl_call_data *) palloc0(sizeof(plperl_call_data));
+       current_call_data->fcinfo = fcinfo;
 
-       /* Set up error handling */
-       memcpy(&save_restart, &Warn_restart, sizeof(save_restart));
+       /* Connect to SPI manager */
+       if (SPI_connect() != SPI_OK_CONNECT)
+               elog(ERROR, "could not connect to SPI manager");
 
-       if (sigsetjmp(Warn_restart, 1) != 0)
-       {
-               memcpy(&Warn_restart, &save_restart, sizeof(Warn_restart));
-               plperl_restart_in_progress = 1;
-               if (--plperl_call_level == 0)
-                       plperl_restart_in_progress = 0;
-               siglongjmp(Warn_restart, 1);
-       }
+       /* Find or compile the function */
+       prodesc = compile_plperl_function(fcinfo->flinfo->fn_oid, true);
+       current_call_data->prodesc = prodesc;
 
-       /************************************************************
-        * Call the Perl function
-        ************************************************************/
-       perlret = plperl_call_perl_func(prodesc, fcinfo);
+       svTD = plperl_trigger_build_args(fcinfo);
+       perlret = plperl_call_perl_trigger_func(prodesc, fcinfo, svTD);
+       hvTD = (HV *) SvRV(svTD);
 
        /************************************************************
-        * Disconnect from SPI manager and then create the return
-        * values datum (if the input function does a palloc for it
-        * this must not be allocated in the SPI memory context
-        * because SPI_finish would free it).
-        ************************************************************/
+       * Disconnect from SPI manager and then create the return
+       * values datum (if the input function does a palloc for it
+       * this must not be allocated in the SPI memory context
+       * because SPI_finish would free it).
+       ************************************************************/
        if (SPI_finish() != SPI_OK_FINISH)
-               elog(ERROR, "plperl: SPI_finish() failed");
+               elog(ERROR, "SPI_finish() failed");
 
-       if (!(perlret && SvOK(perlret)))
+       if (!(perlret && SvOK(perlret) && SvTYPE(perlret) != SVt_NULL))
        {
-               /* return NULL if Perl code returned undef */
-               retval = (Datum) 0;
-               fcinfo->isnull = true;
+               /* undef result means go ahead with original tuple */
+               TriggerData *trigdata = ((TriggerData *) fcinfo->context);
+
+               if (TRIGGER_FIRED_BY_INSERT(trigdata->tg_event))
+                       retval = (Datum) trigdata->tg_trigtuple;
+               else if (TRIGGER_FIRED_BY_UPDATE(trigdata->tg_event))
+                       retval = (Datum) trigdata->tg_newtuple;
+               else if (TRIGGER_FIRED_BY_DELETE(trigdata->tg_event))
+                       retval = (Datum) trigdata->tg_trigtuple;
+               else
+                       retval = (Datum) 0; /* can this happen? */
        }
        else
        {
-               retval = FunctionCall3(&prodesc->result_in_func,
-                                                          PointerGetDatum(SvPV(perlret, PL_na)),
-                                                          ObjectIdGetDatum(prodesc->result_in_elem),
-                                                          Int32GetDatum(-1));
-       }
+               HeapTuple       trv;
+               char       *tmp;
 
-       SvREFCNT_dec(perlret);
+               tmp = SvPV(perlret, PL_na);
 
-       memcpy(&Warn_restart, &save_restart, sizeof(Warn_restart));
-       if (plperl_restart_in_progress)
-       {
-               if (--plperl_call_level == 0)
-                       plperl_restart_in_progress = 0;
-               siglongjmp(Warn_restart, 1);
+               if (pg_strcasecmp(tmp, "SKIP") == 0)
+                       trv = NULL;
+               else if (pg_strcasecmp(tmp, "MODIFY") == 0)
+               {
+                       TriggerData *trigdata = (TriggerData *) fcinfo->context;
+
+                       if (TRIGGER_FIRED_BY_INSERT(trigdata->tg_event))
+                               trv = plperl_modify_tuple(hvTD, trigdata,
+                                                                                 trigdata->tg_trigtuple);
+                       else if (TRIGGER_FIRED_BY_UPDATE(trigdata->tg_event))
+                               trv = plperl_modify_tuple(hvTD, trigdata,
+                                                                                 trigdata->tg_newtuple);
+                       else
+                       {
+                               ereport(WARNING,
+                                               (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                                          errmsg("ignoring modified tuple in DELETE trigger")));
+                               trv = NULL;
+                       }
+               }
+               else
+               {
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                                        errmsg("result of Perl trigger function must be undef, "
+                                                       "\"SKIP\" or \"MODIFY\"")));
+                       trv = NULL;
+               }
+               retval = PointerGetDatum(trv);
        }
 
+       SvREFCNT_dec(svTD);
+       if (perlret)
+               SvREFCNT_dec(perlret);
+
+       current_call_data = NULL;
        return retval;
 }
 
 
-/**********************************************************************
- * compile_plperl_function     - compile (or hopefully just look up) function
- **********************************************************************/
 static plperl_proc_desc *
 compile_plperl_function(Oid fn_oid, bool is_trigger)
 {
@@ -519,33 +1274,35 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
        int                     proname_len;
        plperl_proc_desc *prodesc = NULL;
        int                     i;
+       SV                **svp;
 
        /* We'll need the pg_proc tuple in any case... */
        procTup = SearchSysCache(PROCOID,
                                                         ObjectIdGetDatum(fn_oid),
                                                         0, 0, 0);
        if (!HeapTupleIsValid(procTup))
-               elog(ERROR, "plperl: cache lookup for proc %u failed", fn_oid);
+               elog(ERROR, "cache lookup failed for function %u", fn_oid);
        procStruct = (Form_pg_proc) GETSTRUCT(procTup);
 
        /************************************************************
-        * Build our internal proc name from the functions Oid
+        * Build our internal proc name from the function's Oid
         ************************************************************/
        if (!is_trigger)
                sprintf(internal_proname, "__PLPerl_proc_%u", fn_oid);
        else
                sprintf(internal_proname, "__PLPerl_proc_%u_trigger", fn_oid);
+
        proname_len = strlen(internal_proname);
 
        /************************************************************
         * Lookup the internal proc name in the hashtable
         ************************************************************/
-       if (hv_exists(plperl_proc_hash, internal_proname, proname_len))
+       svp = hv_fetch(plperl_proc_hash, internal_proname, proname_len, FALSE);
+       if (svp)
        {
                bool            uptodate;
 
-               prodesc = (plperl_proc_desc *) SvIV(*hv_fetch(plperl_proc_hash,
-                                                                         internal_proname, proname_len, 0));
+               prodesc = (plperl_proc_desc *) SvIV(*svp);
 
                /************************************************************
                 * If it's present, must check whether it's still up to date.
@@ -553,7 +1310,7 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                 * function's pg_proc entry without changing its OID.
                 ************************************************************/
                uptodate = (prodesc->fn_xmin == HeapTupleHeaderGetXmin(procTup->t_data) &&
-                                       prodesc->fn_cmin == HeapTupleHeaderGetCmin(procTup->t_data));
+                               prodesc->fn_cmin == HeapTupleHeaderGetCmin(procTup->t_data));
 
                if (!uptodate)
                {
@@ -564,7 +1321,7 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
 
        /************************************************************
         * If we haven't found it in the hashtable, we analyze
-        * the functions arguments and returntype and store
+        * the function's arguments and return type and store
         * the in-/out-functions in the prodesc block and create
         * a new hashtable entry for it.
         *
@@ -576,6 +1333,8 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                HeapTuple       typeTup;
                Form_pg_language langStruct;
                Form_pg_type typeStruct;
+               Datum           prosrcdatum;
+               bool            isnull;
                char       *proc_source;
 
                /************************************************************
@@ -583,12 +1342,18 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                 ************************************************************/
                prodesc = (plperl_proc_desc *) malloc(sizeof(plperl_proc_desc));
                if (prodesc == NULL)
-                       elog(ERROR, "plperl: out of memory");
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_OUT_OF_MEMORY),
+                                        errmsg("out of memory")));
                MemSet(prodesc, 0, sizeof(plperl_proc_desc));
                prodesc->proname = strdup(internal_proname);
                prodesc->fn_xmin = HeapTupleHeaderGetXmin(procTup->t_data);
                prodesc->fn_cmin = HeapTupleHeaderGetCmin(procTup->t_data);
 
+               /* Remember if function is STABLE/IMMUTABLE */
+               prodesc->fn_readonly =
+                       (procStruct->provolatile != PROVOLATILE_VOLATILE);
+
                /************************************************************
                 * Lookup the pg_language tuple by Oid
                 ************************************************************/
@@ -599,7 +1364,7 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                {
                        free(prodesc->proname);
                        free(prodesc);
-                       elog(ERROR, "plperl: cache lookup for language %u failed",
+                       elog(ERROR, "cache lookup failed for language %u",
                                 procStruct->prolang);
                }
                langStruct = (Form_pg_language) GETSTRUCT(langTup);
@@ -613,30 +1378,53 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                if (!is_trigger)
                {
                        typeTup = SearchSysCache(TYPEOID,
-                                                               ObjectIdGetDatum(procStruct->prorettype),
+                                                                        ObjectIdGetDatum(procStruct->prorettype),
                                                                         0, 0, 0);
                        if (!HeapTupleIsValid(typeTup))
                        {
                                free(prodesc->proname);
                                free(prodesc);
-                               if (!OidIsValid(procStruct->prorettype))
-                                       elog(ERROR, "plperl functions cannot return type \"opaque\""
-                                                "\n\texcept when used as triggers");
-                               else
-                                       elog(ERROR, "plperl: cache lookup for return type %u failed",
-                                                procStruct->prorettype);
+                               elog(ERROR, "cache lookup failed for type %u",
+                                        procStruct->prorettype);
                        }
                        typeStruct = (Form_pg_type) GETSTRUCT(typeTup);
 
-                       if (typeStruct->typrelid != InvalidOid)
+                       /* Disallow pseudotype result, except VOID or RECORD */
+                       if (typeStruct->typtype == 'p')
                        {
-                               free(prodesc->proname);
-                               free(prodesc);
-                               elog(ERROR, "plperl: return types of tuples not supported yet");
+                               if (procStruct->prorettype == VOIDOID ||
+                                       procStruct->prorettype == RECORDOID)
+                                        /* okay */ ;
+                               else if (procStruct->prorettype == TRIGGEROID)
+                               {
+                                       free(prodesc->proname);
+                                       free(prodesc);
+                                       ereport(ERROR,
+                                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                                        errmsg("trigger functions may only be called "
+                                                                       "as triggers")));
+                               }
+                               else
+                               {
+                                       free(prodesc->proname);
+                                       free(prodesc);
+                                       ereport(ERROR,
+                                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                                        errmsg("plperl functions cannot return type %s",
+                                                                       format_type_be(procStruct->prorettype))));
+                               }
                        }
 
+                       prodesc->result_oid = procStruct->prorettype;
+                       prodesc->fn_retisset = procStruct->proretset;
+                       prodesc->fn_retistuple = (typeStruct->typtype == 'c' ||
+                                                                         procStruct->prorettype == RECORDOID);
+
+                       prodesc->fn_retisarray =
+                               (typeStruct->typlen == -1 && typeStruct->typelem);
+
                        perm_fmgr_info(typeStruct->typinput, &(prodesc->result_in_func));
-                       prodesc->result_in_elem = typeStruct->typelem;
+                       prodesc->result_typioparam = getTypeIOParam(typeTup);
 
                        ReleaseSysCache(typeTup);
                }
@@ -651,27 +1439,37 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                        for (i = 0; i < prodesc->nargs; i++)
                        {
                                typeTup = SearchSysCache(TYPEOID,
-                                                       ObjectIdGetDatum(procStruct->proargtypes[i]),
+                                                ObjectIdGetDatum(procStruct->proargtypes.values[i]),
                                                                                 0, 0, 0);
                                if (!HeapTupleIsValid(typeTup))
                                {
                                        free(prodesc->proname);
                                        free(prodesc);
-                                       if (!OidIsValid(procStruct->proargtypes[i]))
-                                               elog(ERROR, "plperl functions cannot take type \"opaque\"");
-                                       else
-                                               elog(ERROR, "plperl: cache lookup for argument type %u failed",
-                                                        procStruct->proargtypes[i]);
+                                       elog(ERROR, "cache lookup failed for type %u",
+                                                procStruct->proargtypes.values[i]);
                                }
                                typeStruct = (Form_pg_type) GETSTRUCT(typeTup);
 
-                               if (typeStruct->typrelid != InvalidOid)
-                                       prodesc->arg_is_rel[i] = 1;
+                               /* Disallow pseudotype argument */
+                               if (typeStruct->typtype == 'p')
+                               {
+                                       free(prodesc->proname);
+                                       free(prodesc);
+                                       ereport(ERROR,
+                                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                                        errmsg("plperl functions cannot take type %s",
+                                               format_type_be(procStruct->proargtypes.values[i]))));
+                               }
+
+                               if (typeStruct->typtype == 'c')
+                                       prodesc->arg_is_rowtype[i] = true;
                                else
-                                       prodesc->arg_is_rel[i] = 0;
+                               {
+                                       prodesc->arg_is_rowtype[i] = false;
+                                       perm_fmgr_info(typeStruct->typoutput,
+                                                                  &(prodesc->arg_out_func[i]));
+                               }
 
-                               perm_fmgr_info(typeStruct->typoutput, &(prodesc->arg_out_func[i]));
-                               prodesc->arg_out_elem[i] = typeStruct->typelem;
                                ReleaseSysCache(typeTup);
                        }
                }
@@ -680,27 +1478,27 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
                 * create the text of the anonymous subroutine.
                 * we do not use a named subroutine so that we can call directly
                 * through the reference.
-                *
                 ************************************************************/
+               prosrcdatum = SysCacheGetAttr(PROCOID, procTup,
+                                                                         Anum_pg_proc_prosrc, &isnull);
+               if (isnull)
+                       elog(ERROR, "null prosrc");
                proc_source = DatumGetCString(DirectFunctionCall1(textout,
-                                                                 PointerGetDatum(&procStruct->prosrc)));
+                                                                                                                 prosrcdatum));
 
                /************************************************************
                 * Create the procedure in the interpreter
                 ************************************************************/
                prodesc->reference = plperl_create_sub(proc_source, prodesc->lanpltrusted);
                pfree(proc_source);
-               if (!prodesc->reference)
+               if (!prodesc->reference)        /* can this happen? */
                {
                        free(prodesc->proname);
                        free(prodesc);
-                       elog(ERROR, "plperl: cannot create internal procedure %s",
+                       elog(ERROR, "could not create internal procedure \"%s\"",
                                 internal_proname);
                }
 
-               /************************************************************
-                * Add the proc description block to the hashtable
-                ************************************************************/
                hv_store(plperl_proc_hash, internal_proname, proname_len,
                                 newSViv((IV) prodesc), 0);
        }
@@ -711,73 +1509,439 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
 }
 
 
-/**********************************************************************
- * plperl_build_tuple_argument() - Build a string for a ref to a hash
- *                               from all attributes of a given tuple
- **********************************************************************/
+/* Build a hash from all attributes of a given tuple. */
+
 static SV  *
-plperl_build_tuple_argument(HeapTuple tuple, TupleDesc tupdesc)
+plperl_hash_from_tuple(HeapTuple tuple, TupleDesc tupdesc)
 {
+       HV                 *hv;
        int                     i;
-       SV                 *output;
-       Datum           attr;
-       bool            isnull;
-       char       *attname;
-       char       *outputstr;
-       HeapTuple       typeTup;
-       Oid                     typoutput;
-       Oid                     typelem;
 
-       output = sv_2mortal(newSVpv("{", 0));
+       hv = newHV();
 
        for (i = 0; i < tupdesc->natts; i++)
        {
-               /************************************************************
-                * Get the attribute name
-                ************************************************************/
-               attname = tupdesc->attrs[i]->attname.data;
+               Datum           attr;
+               bool            isnull;
+               char       *attname;
+               char       *outputstr;
+               Oid                     typoutput;
+               bool            typisvarlena;
+               int                     namelen;
+               SV                 *sv;
+
+               if (tupdesc->attrs[i]->attisdropped)
+                       continue;
 
-               /************************************************************
-                * Get the attributes value
-                ************************************************************/
+               attname = NameStr(tupdesc->attrs[i]->attname);
+               namelen = strlen(attname);
                attr = heap_getattr(tuple, i + 1, tupdesc, &isnull);
 
-               /************************************************************
-                *      If it is null it will be set to undef in the hash.
-                ************************************************************/
                if (isnull)
                {
-                       sv_catpvf(output, "'%s' => undef,", attname);
+                       /* Store (attname => undef) and move on. */
+                       hv_store(hv, attname, namelen, newSV(0), 0);
                        continue;
                }
 
-               /************************************************************
-                * Lookup the attribute type in the syscache
-                * for the output function
-                ************************************************************/
-               typeTup = SearchSysCache(TYPEOID,
-                                                  ObjectIdGetDatum(tupdesc->attrs[i]->atttypid),
-                                                                0, 0, 0);
-               if (!HeapTupleIsValid(typeTup))
-                       elog(ERROR, "plperl: Cache lookup for attribute '%s' type %u failed",
-                                attname, tupdesc->attrs[i]->atttypid);
+               /* XXX should have a way to cache these lookups */
 
-               typoutput = ((Form_pg_type) GETSTRUCT(typeTup))->typoutput;
-               typelem = ((Form_pg_type) GETSTRUCT(typeTup))->typelem;
-               ReleaseSysCache(typeTup);
+               getTypeOutputInfo(tupdesc->attrs[i]->atttypid,
+                                                 &typoutput, &typisvarlena);
+
+               outputstr = DatumGetCString(OidFunctionCall1(typoutput, attr));
+
+               sv = newSVpv(outputstr, 0);
+#if PERL_BCDVERSION >= 0x5006000L
+               if (GetDatabaseEncoding() == PG_UTF8)
+                       SvUTF8_on(sv);
+#endif
+               hv_store(hv, attname, namelen, sv, 0);
 
-               /************************************************************
-                * Append the attribute name and the value to the list.
-                ************************************************************/
-               outputstr = DatumGetCString(OidFunctionCall3(typoutput,
-                                                                                                        attr,
-                                                                                          ObjectIdGetDatum(typelem),
-                                                  Int32GetDatum(tupdesc->attrs[i]->atttypmod)));
-               sv_catpvf(output, "'%s' => '%s',", attname, outputstr);
                pfree(outputstr);
        }
 
-       sv_catpv(output, "}");
-       output = perl_eval_pv(SvPV(output, PL_na), TRUE);
-       return output;
+       return newRV_noinc((SV *) hv);
+}
+
+
+HV *
+plperl_spi_exec(char *query, int limit)
+{
+       HV                 *ret_hv;
+
+       /*
+        * Execute the query inside a sub-transaction, so we can cope with errors
+        * sanely
+        */
+       MemoryContext oldcontext = CurrentMemoryContext;
+       ResourceOwner oldowner = CurrentResourceOwner;
+
+       BeginInternalSubTransaction(NULL);
+       /* Want to run inside function's memory context */
+       MemoryContextSwitchTo(oldcontext);
+
+       PG_TRY();
+       {
+               int                     spi_rv;
+
+               spi_rv = SPI_execute(query, current_call_data->prodesc->fn_readonly,
+                                                        limit);
+               ret_hv = plperl_spi_execute_fetch_result(SPI_tuptable, SPI_processed,
+                                                                                                spi_rv);
+
+               /* Commit the inner transaction, return to outer xact context */
+               ReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * AtEOSubXact_SPI() should not have popped any SPI context, but just
+                * in case it did, make sure we remain connected.
+                */
+               SPI_restore_connection();
+       }
+       PG_CATCH();
+       {
+               ErrorData  *edata;
+
+               /* Save error info */
+               MemoryContextSwitchTo(oldcontext);
+               edata = CopyErrorData();
+               FlushErrorState();
+
+               /* Abort the inner transaction */
+               RollbackAndReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * If AtEOSubXact_SPI() popped any SPI context of the subxact, it will
+                * have left us in a disconnected state.  We need this hack to return
+                * to connected state.
+                */
+               SPI_restore_connection();
+
+               /* Punt the error to Perl */
+               croak("%s", edata->message);
+
+               /* Can't get here, but keep compiler quiet */
+               return NULL;
+       }
+       PG_END_TRY();
+
+       return ret_hv;
+}
+
+
+static HV  *
+plperl_spi_execute_fetch_result(SPITupleTable *tuptable, int processed,
+                                                               int status)
+{
+       HV                 *result;
+
+       result = newHV();
+
+       hv_store(result, "status", strlen("status"),
+                        newSVpv((char *) SPI_result_code_string(status), 0), 0);
+       hv_store(result, "processed", strlen("processed"),
+                        newSViv(processed), 0);
+
+       if (status == SPI_OK_SELECT)
+       {
+               AV                 *rows;
+               SV                 *row;
+               int                     i;
+
+               rows = newAV();
+               for (i = 0; i < processed; i++)
+               {
+                       row = plperl_hash_from_tuple(tuptable->vals[i], tuptable->tupdesc);
+                       av_push(rows, row);
+               }
+               hv_store(result, "rows", strlen("rows"),
+                                newRV_noinc((SV *) rows), 0);
+       }
+
+       SPI_freetuptable(tuptable);
+
+       return result;
+}
+
+
+/*
+ * Note: plperl_return_next is called both in Postgres and Perl contexts.
+ * We report any errors in Postgres fashion (via ereport).     If called in
+ * Perl context, it is SPI.xs's responsibility to catch the error and
+ * convert to a Perl error.  We assume (perhaps without adequate justification)
+ * that we need not abort the current transaction if the Perl code traps the
+ * error.
+ */
+void
+plperl_return_next(SV *sv)
+{
+       plperl_proc_desc *prodesc;
+       FunctionCallInfo fcinfo;
+       ReturnSetInfo *rsi;
+       MemoryContext old_cxt;
+       HeapTuple       tuple;
+
+       if (!sv)
+               return;
+
+       prodesc = current_call_data->prodesc;
+       fcinfo = current_call_data->fcinfo;
+       rsi = (ReturnSetInfo *) fcinfo->resultinfo;
+
+       if (!prodesc->fn_retisset)
+               ereport(ERROR,
+                               (errcode(ERRCODE_SYNTAX_ERROR),
+                                errmsg("cannot use return_next in a non-SETOF function")));
+
+       if (prodesc->fn_retistuple &&
+               !(SvOK(sv) && SvTYPE(sv) == SVt_RV && SvTYPE(SvRV(sv)) == SVt_PVHV))
+               ereport(ERROR,
+                               (errcode(ERRCODE_DATATYPE_MISMATCH),
+                                errmsg("setof-composite-returning Perl function "
+                                               "must call return_next with reference to hash")));
+
+       if (!current_call_data->ret_tdesc)
+       {
+               TupleDesc tupdesc;
+
+               Assert(!current_call_data->tuple_store);
+               Assert(!current_call_data->attinmeta);
+
+               /*
+                * This is the first call to return_next in the current
+                * PL/Perl function call, so memoize some lookups
+                */
+               if (prodesc->fn_retistuple)
+                       (void) get_call_result_type(fcinfo, NULL, &tupdesc);
+               else
+                       tupdesc = rsi->expectedDesc;
+
+               /*
+                * Make sure the tuple_store and ret_tdesc are sufficiently
+                * long-lived.
+                */
+               old_cxt = MemoryContextSwitchTo(rsi->econtext->ecxt_per_query_memory);
+
+               current_call_data->ret_tdesc = CreateTupleDescCopy(tupdesc);
+               current_call_data->tuple_store =
+                       tuplestore_begin_heap(true, false, work_mem);
+               if (prodesc->fn_retistuple)
+               {
+                       current_call_data->attinmeta =
+                               TupleDescGetAttInMetadata(current_call_data->ret_tdesc);
+               }
+
+               MemoryContextSwitchTo(old_cxt);
+       }               
+
+       /*
+        * Producing the tuple we want to return requires making plenty of
+        * palloc() allocations that are not cleaned up. Since this
+        * function can be called many times before the current memory
+        * context is reset, we need to do those allocations in a
+        * temporary context.
+        */
+       if (!current_call_data->tmp_cxt)
+       {
+               current_call_data->tmp_cxt =
+                       AllocSetContextCreate(rsi->econtext->ecxt_per_tuple_memory,
+                                                                 "PL/Perl return_next temporary cxt",
+                                                                 ALLOCSET_DEFAULT_MINSIZE,
+                                                                 ALLOCSET_DEFAULT_INITSIZE,
+                                                                 ALLOCSET_DEFAULT_MAXSIZE);
+       }
+
+       old_cxt = MemoryContextSwitchTo(current_call_data->tmp_cxt);
+
+       if (prodesc->fn_retistuple)
+       {
+               tuple = plperl_build_tuple_result((HV *) SvRV(sv),
+                                                                                 current_call_data->attinmeta);
+       }
+       else
+       {
+               Datum           ret = (Datum) 0;
+               bool            isNull = true;
+
+               if (SvOK(sv) && SvTYPE(sv) != SVt_NULL)
+               {
+                       char       *val = SvPV(sv, PL_na);
+
+                       ret = FunctionCall3(&prodesc->result_in_func,
+                                                               PointerGetDatum(val),
+                                                               ObjectIdGetDatum(prodesc->result_typioparam),
+                                                               Int32GetDatum(-1));
+                       isNull = false;
+               }
+
+               tuple = heap_form_tuple(current_call_data->ret_tdesc, &ret, &isNull);
+       }
+
+       /* Make sure to store the tuple in a long-lived memory context */
+       MemoryContextSwitchTo(rsi->econtext->ecxt_per_query_memory);
+       tuplestore_puttuple(current_call_data->tuple_store, tuple);
+       MemoryContextSwitchTo(old_cxt);
+
+       MemoryContextReset(current_call_data->tmp_cxt);
+}
+
+
+SV *
+plperl_spi_query(char *query)
+{
+       SV                 *cursor;
+
+       /*
+        * Execute the query inside a sub-transaction, so we can cope with errors
+        * sanely
+        */
+       MemoryContext oldcontext = CurrentMemoryContext;
+       ResourceOwner oldowner = CurrentResourceOwner;
+
+       BeginInternalSubTransaction(NULL);
+       /* Want to run inside function's memory context */
+       MemoryContextSwitchTo(oldcontext);
+
+       PG_TRY();
+       {
+               void       *plan;
+               Portal          portal = NULL;
+
+               /* Create a cursor for the query */
+               plan = SPI_prepare(query, 0, NULL);
+               if (plan)
+                       portal = SPI_cursor_open(NULL, plan, NULL, NULL, false);
+               if (portal)
+                       cursor = newSVpv(portal->name, 0);
+               else
+                       cursor = newSV(0);
+
+               /* Commit the inner transaction, return to outer xact context */
+               ReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * AtEOSubXact_SPI() should not have popped any SPI context, but just
+                * in case it did, make sure we remain connected.
+                */
+               SPI_restore_connection();
+       }
+       PG_CATCH();
+       {
+               ErrorData  *edata;
+
+               /* Save error info */
+               MemoryContextSwitchTo(oldcontext);
+               edata = CopyErrorData();
+               FlushErrorState();
+
+               /* Abort the inner transaction */
+               RollbackAndReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * If AtEOSubXact_SPI() popped any SPI context of the subxact, it will
+                * have left us in a disconnected state.  We need this hack to return
+                * to connected state.
+                */
+               SPI_restore_connection();
+
+               /* Punt the error to Perl */
+               croak("%s", edata->message);
+
+               /* Can't get here, but keep compiler quiet */
+               return NULL;
+       }
+       PG_END_TRY();
+
+       return cursor;
+}
+
+
+SV *
+plperl_spi_fetchrow(char *cursor)
+{
+       SV                 *row;
+
+       /*
+        * Execute the FETCH inside a sub-transaction, so we can cope with errors
+        * sanely
+        */
+       MemoryContext oldcontext = CurrentMemoryContext;
+       ResourceOwner oldowner = CurrentResourceOwner;
+
+       BeginInternalSubTransaction(NULL);
+       /* Want to run inside function's memory context */
+       MemoryContextSwitchTo(oldcontext);
+
+       PG_TRY();
+       {
+               Portal          p = SPI_cursor_find(cursor);
+
+               if (!p)
+                       row = newSV(0);
+               else
+               {
+                       SPI_cursor_fetch(p, true, 1);
+                       if (SPI_processed == 0)
+                       {
+                               SPI_cursor_close(p);
+                               row = newSV(0);
+                       }
+                       else
+                       {
+                               row = plperl_hash_from_tuple(SPI_tuptable->vals[0],
+                                                                                        SPI_tuptable->tupdesc);
+                       }
+                       SPI_freetuptable(SPI_tuptable);
+               }
+
+               /* Commit the inner transaction, return to outer xact context */
+               ReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * AtEOSubXact_SPI() should not have popped any SPI context, but just
+                * in case it did, make sure we remain connected.
+                */
+               SPI_restore_connection();
+       }
+       PG_CATCH();
+       {
+               ErrorData  *edata;
+
+               /* Save error info */
+               MemoryContextSwitchTo(oldcontext);
+               edata = CopyErrorData();
+               FlushErrorState();
+
+               /* Abort the inner transaction */
+               RollbackAndReleaseCurrentSubTransaction();
+               MemoryContextSwitchTo(oldcontext);
+               CurrentResourceOwner = oldowner;
+
+               /*
+                * If AtEOSubXact_SPI() popped any SPI context of the subxact, it will
+                * have left us in a disconnected state.  We need this hack to return
+                * to connected state.
+                */
+               SPI_restore_connection();
+
+               /* Punt the error to Perl */
+               croak("%s", edata->message);
+
+               /* Can't get here, but keep compiler quiet */
+               return NULL;
+       }
+       PG_END_TRY();
+
+       return row;
 }