]> granicus.if.org Git - postgresql/commitdiff
Back out SQLSTATE and SQLERRM support.
authorBruce Momjian <bruce@momjian.us>
Thu, 26 May 2005 04:08:32 +0000 (04:08 +0000)
committerBruce Momjian <bruce@momjian.us>
Thu, 26 May 2005 04:08:32 +0000 (04:08 +0000)
doc/src/sgml/plpgsql.sgml
src/pl/plpgsql/src/gram.y
src/pl/plpgsql/src/pl_exec.c
src/pl/plpgsql/src/plpgsql.h
src/test/regress/expected/plpgsql.out
src/test/regress/sql/plpgsql.sql

index 17e62744c5e7665298ab825caa78284e05fdf660..7d2b37e41d7b801b20e2e5c41a0544bef14df2ed 100644 (file)
@@ -1,5 +1,5 @@
 <!--
-$PostgreSQL: pgsql/doc/src/sgml/plpgsql.sgml,v 1.68 2005/05/26 00:16:31 momjian Exp $
+$PostgreSQL: pgsql/doc/src/sgml/plpgsql.sgml,v 1.69 2005/05/26 04:08:31 momjian Exp $
 -->
 
 <chapter id="plpgsql"> 
@@ -2007,13 +2007,12 @@ END LOOP;
     </indexterm>
 
     <para>
-     Any error occurring in <application>PL/pgSQL</> sets variables
-     <varname>SQLSTATE</> and <varname>SQLERRM</>, and, by default,
-     aborts execution of the function, and indeed of the surrounding
-     transaction as well. You can trap errors and recover from them by
-     using a <command>BEGIN</> block with an <literal>EXCEPTION</>
-     clause. The syntax is an extension of the normal syntax for a
-     <command>BEGIN</> block:
+     By default, any error occurring in a <application>PL/pgSQL</>
+     function aborts execution of the function, and indeed of the
+     surrounding transaction as well.  You can trap errors and recover
+     from them by using a <command>BEGIN</> block with an
+     <literal>EXCEPTION</> clause.  The syntax is an extension of the
+     normal syntax for a <command>BEGIN</> block:
 
 <synopsis>
 <optional> &lt;&lt;<replaceable>label</replaceable>&gt;&gt; </optional>
index 39a7a01bebe3fc37e808fcc167de98f6bad77d0d..028323192fa394a3987afc08a182d4428afee8de 100644 (file)
@@ -4,7 +4,7 @@
  *                                               procedural language
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/pl/plpgsql/src/gram.y,v 1.71 2005/05/26 03:18:53 neilc Exp $
+ *       $PostgreSQL: pgsql/src/pl/plpgsql/src/gram.y,v 1.72 2005/05/26 04:08:31 momjian Exp $
  *
  *       This software is copyrighted by Jan Wieck - Hamburg.
  *
@@ -80,11 +80,6 @@ static       void                     plpgsql_sql_error_callback(void *arg);
                        int  n_initvars;
                        int  *initvarnos;
                }                                               declhdr;
-               struct 
-               {
-                       int sqlstate_varno;
-                       int sqlerrm_varno;
-               }                                               fict_vars;
                List                                    *list;
                PLpgSQL_type                    *dtype;
                PLpgSQL_datum                   *scalar;        /* a VAR, RECFIELD, or TRIGARG */
@@ -101,7 +96,6 @@ static       void                     plpgsql_sql_error_callback(void *arg);
                PLpgSQL_diag_item               *diagitem;
 }
 
-%type <fict_vars> fict_vars_sect
 %type <declhdr> decl_sect
 %type <varname> decl_varname
 %type <str>            decl_renname
@@ -250,22 +244,19 @@ opt_semi          :
                                | ';'
                                ;
 
-pl_block               : decl_sect fict_vars_sect K_BEGIN lno proc_sect exception_sect K_END
+pl_block               : decl_sect K_BEGIN lno proc_sect exception_sect K_END
                                        {
                                                PLpgSQL_stmt_block *new;
 
                                                new = palloc0(sizeof(PLpgSQL_stmt_block));
 
                                                new->cmd_type   = PLPGSQL_STMT_BLOCK;
-                                               new->lineno             = $4;
+                                               new->lineno             = $3;
                                                new->label              = $1.label;
                                                new->n_initvars = $1.n_initvars;
                                                new->initvarnos = $1.initvarnos;
-                                               new->body               = $5;
-                                               new->exceptions = $6;
-
-                                               new->sqlstate_varno = $2.sqlstate_varno;
-                                               new->sqlerrm_varno = $2.sqlerrm_varno;
+                                               new->body               = $4;
+                                               new->exceptions = $5;
 
                                                plpgsql_ns_pop();
 
@@ -273,20 +264,6 @@ pl_block           : decl_sect fict_vars_sect K_BEGIN lno proc_sect exception_sect K_END
                                        }
                                ;
 
-fict_vars_sect :
-                                       {
-                                               PLpgSQL_variable        *var;
-
-                                               plpgsql_ns_setlocal(false);
-                                               var = plpgsql_build_variable("sqlstate", 0,
-                                                                                plpgsql_build_datatype(TEXTOID, -1), true);  
-                                               $$.sqlstate_varno = var->dno;
-                                               var = plpgsql_build_variable("sqlerrm", 0,
-                                                                                plpgsql_build_datatype(TEXTOID, -1), true);  
-                                               $$.sqlerrm_varno = var->dno;
-                                               plpgsql_add_initdatums(NULL);
-                                       }
-                               ;
 
 decl_sect              : opt_label
                                        {
index c602da21601f7b196b89aa541c27324dbe3b20c3..3c6216a3e56c797ad197030bd0cd0cce618f3428 100644 (file)
@@ -3,7 +3,7 @@
  *                       procedural language
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.140 2005/05/26 03:18:53 neilc Exp $
+ *       $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.141 2005/05/26 04:08:31 momjian Exp $
  *
  *       This software is copyrighted by Jan Wieck - Hamburg.
  *
@@ -180,7 +180,6 @@ static Datum exec_simple_cast_value(Datum value, Oid valtype,
 static void exec_init_tuple_store(PLpgSQL_execstate *estate);
 static bool compatible_tupdesc(TupleDesc td1, TupleDesc td2);
 static void exec_set_found(PLpgSQL_execstate *estate, bool state);
-static char *unpack_sql_state(int ssval);
 
 
 /* ----------
@@ -748,20 +747,6 @@ exec_stmt_block(PLpgSQL_execstate *estate, PLpgSQL_stmt_block *block)
        int                     i;
        int                     n;
 
-
-       /* setup SQLSTATE and SQLERRM */
-       PLpgSQL_var *var;
-  
-       var = (PLpgSQL_var *) (estate->datums[block->sqlstate_varno]);
-       var->isnull = false;
-       var->freeval = true;
-       var->value = DirectFunctionCall1(textin, CStringGetDatum("00000"));
-  
-       var = (PLpgSQL_var *) (estate->datums[block->sqlerrm_varno]);
-       var->isnull = false;
-       var->freeval = true;
-       var->value = DirectFunctionCall1(textin, CStringGetDatum("Successful completion"));
-
        /*
         * First initialize all variables declared in this block
         */
@@ -777,7 +762,7 @@ exec_stmt_block(PLpgSQL_execstate *estate, PLpgSQL_stmt_block *block)
 
                                        if (var->freeval)
                                        {
-                                               pfree(DatumGetPointer(var->value));
+                                               pfree((void *) (var->value));
                                                var->freeval = false;
                                        }
 
@@ -870,15 +855,6 @@ exec_stmt_block(PLpgSQL_execstate *estate, PLpgSQL_stmt_block *block)
                        RollbackAndReleaseCurrentSubTransaction();
                        MemoryContextSwitchTo(oldcontext);
                        CurrentResourceOwner = oldowner;
-                       /* set SQLSTATE and SQLERRM variables */
-                       var = (PLpgSQL_var *) (estate->datums[block->sqlstate_varno]);
-                       pfree(DatumGetPointer(var->value));
-                       var->value = DirectFunctionCall1(textin, CStringGetDatum(unpack_sql_state(edata->sqlerrcode)));
-
-                       var = (PLpgSQL_var *) (estate->datums[block->sqlerrm_varno]);
-                       pfree(DatumGetPointer(var->value));
-                       var->value = DirectFunctionCall1(textin, CStringGetDatum(edata->message));
 
                        /*
                         * If AtEOSubXact_SPI() popped any SPI context of the subxact,
@@ -943,26 +919,6 @@ exec_stmt_block(PLpgSQL_execstate *estate, PLpgSQL_stmt_block *block)
        return PLPGSQL_RC_OK;
 }
 
-/* 
- * unpack MAKE_SQLSTATE code 
- * This code is copied from backend/utils/error/elog.c.
- */
-static char *
-unpack_sql_state(int ssval)
-{
-       static  char            tbuf[12];
-       int                     i;
-
-       for (i = 0; i < 5; i++)
-       {
-               tbuf[i] = PGUNSIXBIT(ssval);
-               ssval >>= 6;
-       }
-       tbuf[i] = '\0';
-       return tbuf;
-}
-
-
 
 /* ----------
  * exec_stmts                  Iterate over a list of statements
index f3b14fdf643910d56f8edc243fc75f01a58e1bfa..17a466e01e8cd5b4b48de63c7c21f4dba8fd78c5 100644 (file)
@@ -3,7 +3,7 @@
  *                       procedural language
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/pl/plpgsql/src/plpgsql.h,v 1.59 2005/05/26 00:16:31 momjian Exp $
+ *       $PostgreSQL: pgsql/src/pl/plpgsql/src/plpgsql.h,v 1.60 2005/05/26 04:08:31 momjian Exp $
  *
  *       This software is copyrighted by Jan Wieck - Hamburg.
  *
@@ -336,11 +336,9 @@ typedef struct
        int                     lineno;
        char       *label;
        List       *body;                       /* List of statements */
-       List       *exceptions;         /* List of WHEN clauses */
-       int             n_initvars;
-       int        *initvarnos;
-       int             sqlstate_varno;
-       int             sqlerrm_varno;
+       List       *exceptions;         /* List of WHEN clauses */
+       int                     n_initvars;
+       int                *initvarnos;
 } PLpgSQL_stmt_block;
 
 
index aa50550f23f4e162c222fef83726a45f3505d8e9..08fbe46b3a240f4932ff382d3370ed20f44e580d 100644 (file)
@@ -2380,29 +2380,3 @@ ERROR:  control reached end of function without RETURN
 CONTEXT:  PL/pgSQL function "missing_return_expr"
 drop function void_return_expr();
 drop function missing_return_expr();
--- test SQLSTATE and SQLERRM
-create function trap_exceptions() returns void as $_$
-begin
-   begin
-     raise exception 'first exception';
-   exception when others then
-     raise notice '% %', SQLSTATE, SQLERRM;
-   end;
-   raise notice '% %', SQLSTATE, SQLERRM;
-   begin
-     raise exception 'last exception';
-   exception when others then
-     raise notice '% %', SQLSTATE, SQLERRM;
-   end;
-   return;
-end; $_$ language plpgsql;
-select trap_exceptions();
-NOTICE:  P0001 first exception
-NOTICE:  00000 Successful completion
-NOTICE:  P0001 last exception
- trap_exceptions 
------------------
-(1 row)
-
-drop function trap_exceptions();
index 3703587ad16c1cbccfc6d8a9ce9a3c0e923de580..7ea7c8c6e0c558243f82b891059b0a77a848debf 100644 (file)
@@ -2018,24 +2018,3 @@ select missing_return_expr();
 
 drop function void_return_expr();
 drop function missing_return_expr();
-
--- test SQLSTATE and SQLERRM
-create function trap_exceptions() returns void as $_$
-begin
-   begin
-     raise exception 'first exception';
-   exception when others then
-     raise notice '% %', SQLSTATE, SQLERRM;
-   end;
-   raise notice '% %', SQLSTATE, SQLERRM;
-   begin
-     raise exception 'last exception';
-   exception when others then
-     raise notice '% %', SQLSTATE, SQLERRM;
-   end;
-   return;
-end; $_$ language plpgsql;
-
-select trap_exceptions();
-
-drop function trap_exceptions();