Skip site navigation (1) Skip section navigation (2)

Re: Slow PL/pgSQL 8.0.RC5 (7.4.6. 3times faster)

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Mark Kirkwood <markir(at)coretech(dot)co(dot)nz>
Cc: Michael Fuhr <mike(at)fuhr(dot)org>,Pavel Stehule <stehule(at)kix(dot)fsv(dot)cvut(dot)cz>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Slow PL/pgSQL 8.0.RC5 (7.4.6. 3times faster)
Date: 2005-01-14 05:15:28
Message-ID: 19846.1105679728@sss.pgh.pa.us (view raw or flat)
Thread:
Lists: pgsql-hackers
Mark Kirkwood <markir(at)coretech(dot)co(dot)nz> writes:
> Looks like something introduced between RC1 and RC5:

You mean here

2004-12-19 15:20  tgl

	* src/: pl/plpgsql/src/pl_exec.c,
	test/regress/expected/plpgsql.out, test/regress/sql/plpgsql.sql:
	plpgsql's exec_eval_simple_expr() now has to take responsibility
	for advancing ActiveSnapshot when we are inside a volatile
	function.  Per example from Gaetano Mendola.  Add a regression test
	to catch similar problems in future.

and here

2004-12-21 13:33  tgl

	* src/: pl/plpgsql/src/pl_exec.c,
	test/regress/expected/plpgsql.out, test/regress/sql/plpgsql.sql:
	exec_eval_simple_expr() needs to do CommandCounterIncrement() not
	just GetTransactionSnapshot() to ensure ActiveSnapshot advances
	properly.  Sigh.  Extend regression test so it reveals this error
	too.

			regards, tom lane

In response to

Responses

pgsql-hackers by date

Next:From: Marc G. FournierDate: 2005-01-14 05:53:34
Subject: Re: FATAL: catalog is missing 1 attribute(s) for relid
Previous:From: Greg StarkDate: 2005-01-14 04:50:01
Subject: Re: Bug? 8.0 does not use partial index

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group