mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

EXECUTE of a SELECT ... INTO now draws a 'not implemented' error,

Browse Source 
rather than executing the INTO clause with non-plpgsql semantics
as it was doing for the last few weeks/months.  This keeps our options
open for making it do the right plpgsql-ish thing in future without
creating a backwards compatibility problem.  There is no loss of
functionality since people can get the same behavior with CREATE TABLE AS.
This commit is contained in:
Tom Lane 2001-02-09 00:14:26 +00:00
parent bbac19a973
commit 8a2cdd77ad
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/pl/plpgsql/src/pl_exec.c
View File

@ -3,7 +3,7 @@
* procedural language
*
* IDENTIFICATION
* $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.36 2001/01/22 00:50:07 tgl Exp $
* $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.37 2001/02/09 00:14:26 tgl Exp $
*
* This software is copyrighted by Jan Wieck - Hamburg.
*
@ -1993,7 +1993,6 @@ exec_stmt_dynexecute(PLpgSQL_execstate * estate,
switch (exec_res)
{
case SPI_OK_SELECT:
case SPI_OK_SELINTO:
case SPI_OK_INSERT:
case SPI_OK_UPDATE:
case SPI_OK_DELETE:
@ -2006,6 +2005,16 @@ exec_stmt_dynexecute(PLpgSQL_execstate * estate,
*/
break;
case SPI_OK_SELINTO:
/*
* Disallow this for now, because its behavior is not consistent
* with SELECT INTO in a normal plpgsql context. We need to
* reimplement EXECUTE to parse the string as a plpgsql command,
* not just feed it to SPI_exec.
*/
elog(ERROR, "EXECUTE of SELECT ... INTO is not implemented yet");
break;
default:
elog(ERROR, "unexpected error %d in EXECUTE of query '%s'",
exec_res, querystr);