Commit6918df1

committed

plpgsql's PERFORM statement now sets FOUND depending on whether any

rows were returned by the performed query. Per recent pgsql-generaldiscussion.

1 parente11f167 commit6918df1Copy full SHA for 6918df1

File tree

+13

-11

lines changed

+13

-11

lines changed

Lines changed: 5 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`<!--`
`2`		`-$Header: /cvsroot/pgsql/doc/src/sgml/Attic/plsql.sgml,v 2.59 2002/06/15 19:34:51 momjian Exp $`
	`2`	`+$Header: /cvsroot/pgsql/doc/src/sgml/Attic/plsql.sgml,v 2.60 2002/06/24 23:12:06 tgl Exp $`
`3`	`3`	`-->`
`4`	`4`
`5`	`5`	`<chapter id="plpgsql">`
`@@ -903,7 +903,9 @@ PERFORM <replaceable>query</replaceable>;`
`903`	`903`	`This executes a <literal>SELECT</literal>`
`904`	`904`	`<replaceable>query</replaceable> and discards the`
`905`	`905`	`result. <application>PL/pgSQL</application> variables are substituted`
`906`		`- in the query as usual.`
	`906`	`+ in the query as usual. Also, the special variable FOUND is set to`
	`907`	`+ true if the query produced at least one row, or false if it produced`
	`908`	`+ no rows.`
`907`	`909`	`</para>`
`908`	`910`
`909`	`911`	`<note>`
`@@ -916,11 +918,7 @@ PERFORM <replaceable>query</replaceable>;`
`916`	`918`	`<para>`
`917`	`919`	`An example:`
`918`	`920`	`<programlisting>`
`919`		`-PERFORM create_mv(''cs_session_page_requests_mv'',''`
`920`		`- SELECT session_id, page_id, count(*) AS n_hits,`
`921`		`- sum(dwell_time) AS dwell_time, count(dwell_time) AS dwell_count`
`922`		`- FROM cs_fact_table`
`923`		`- GROUP BY session_id, page_id '');`
	`921`	`+PERFORM create_mv(''cs_session_page_requests_mv'', my_query);`
`924`	`922`	`</programlisting>`
`925`	`923`	`</para>`
`926`	`924`	`</sect2>`

Lines changed: 8 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`* procedural language`
`4`	`4`	`*`
`5`	`5`	`* IDENTIFICATION`
`6`		`- * $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.55 2002/03/25 07:41:10 tgl Exp $`
	`6`	`+ * $Header: /cvsroot/pgsql/src/pl/plpgsql/src/pl_exec.c,v 1.56 2002/06/24 23:12:06 tgl Exp $`
`7`	`7`	`*`
`8`	`8`	`* This software is copyrighted by Jan Wieck - Hamburg.`
`9`	`9`	`*`
`@@ -969,9 +969,11 @@ exec_stmt_assign(PLpgSQL_execstate * estate, PLpgSQL_stmt_assign * stmt)`
`969`	`969`	`else`
`970`	`970`	`{`
`971`	`971`	`/*`
`972`		`- * PERFORM: evaluate query and discard result.This cannot share`
`973`		`- * code with the assignment case since we do not wish to`
`974`		`- * constraint the discarded result to be only one row/column.`
	`972`	`+ * PERFORM: evaluate query and discard result (but set FOUND`
	`973`	`+ * depending on whether at least one row was returned).`
	`974`	`+ *`
	`975`	`+ * This cannot share code with the assignment case since we do not`
	`976`	`+ * wish to constrain the discarded result to be only one row/column.`
`975`	`977`	`*/`
`976`	`978`	`intrc;`
`977`	`979`
`@@ -985,6 +987,8 @@ exec_stmt_assign(PLpgSQL_execstate * estate, PLpgSQL_stmt_assign * stmt)`
`985`	`987`	`if (rc!=SPI_OK_SELECT)`
`986`	`988`	`elog(ERROR,"query \"%s\" didn't return data",expr->query);`
`987`	`989`
	`990`	`+exec_set_found(estate, (estate->eval_processed!=0));`
	`991`	`+`
`988`	`992`	`exec_eval_cleanup(estate);`
`989`	`993`	`}`
`990`	`994`

Comments

(0)