Re: Slow function in queries SELECT clause.

From: Tom Lane
Subject: Re: Slow function in queries SELECT clause.
Date: ,
Msg-id: 25116.1277047267@sss.pgh.pa.us
(view: Whole thread, Raw)
In response to: Slow function in queries SELECT clause.  ("Davor J.")
List: pgsql-performance

Tree view

Slow function in queries SELECT clause.  ("Davor J.", )
 Re: Slow function in queries SELECT clause.  (Szymon Guz, )
 Re: Slow function in queries SELECT clause.  ("Davor J.", )
 Re: Slow function in queries SELECT clause.  (Tom Lane, )
 Re: Slow function in queries SELECT clause.  ("Davor J.", )
 Re: Slow function in queries SELECT clause.  ("Davor J.", )

"Davor J." <> writes:
> Suppose 2 functions: factor(int,int) and offset(int, int).
> Suppose a third function: convert(float,int,int) which simply returns
> $1*factor($2,$3)+offset($2,$3)
> All three functions are IMMUTABLE.

You should write the third function as a SQL function, which'd allow it
to be inlined.

> VERY FAST (half a second):
> ----------------
> SELECT data*factor(1,2)+offset(1,2) FROM tbl_data;

In this case both factor() calls are folded to constants, hence executed
only once.

> VERY SLOW (a minute):
> ----------------
> SELECT convert(data, 1, 2) FROM tbl_data;

Without inlining, there's no hope of any constant-folding here.
The optimizer just sees the plpgsql function as a black box and
can't do anything with it.

BTW, your later mail shows that the factor() functions are not really
IMMUTABLE, since they select from tables that presumably are subject to
change.  The "correct" declaration would be STABLE.  If you're relying
on constant-folding to get reasonable application performance, you're
going to have to continue to mislabel them as IMMUTABLE; but be aware
that you're likely to have issues any time you do change the table
contents.  The changes won't get reflected into existing query plans.

            regards, tom lane


pgsql-performance by date:

From: Jesper Krogh
Date:
Subject: Aggressive autovacuuming ?
From: Greg Smith
Date:
Subject: Re: B-Heaps