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
Views: Raw Message | Whole Thread | Download mbox | Resend email
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

Browse pgsql-hackers by date

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