Message-ID: <4ea0957f-d599-47e5-ae9e-758ec6711e03@technowledgy.de>
Date: Mon, 14 Apr 2025 12:18:44 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: Interesting case of IMMUTABLE significantly hurting performance
Content-Language: en-US
To: Tom Lane <tgl@sss.pgh.pa.us>,
 "David G. Johnston" <david.g.johnston@gmail.com>
Cc: Laurenz Albe <laurenz.albe@cybertec.at>, Olleg Samoylov <splarv@ya.ru>,
 "pgsql-generallists.postgresql.org" <pgsql-general@lists.postgresql.org>
References: <662101e2-5ca5-4bf5-9307-2519efde8690@ya.ru>
 <8687df7d49bc4b28b41c7cf4e9ad1e15d383b2fa.camel@cybertec.at>
 <CAKFQuwY51Us6LeBxtdL7MbHpJkxYpWtMOHJAhUoXXN2p5f1JfQ@mail.gmail.com>
 <373227.1744236488@sss.pgh.pa.us>
From: Wolfgang Walther <walther@technowledgy.de>
In-Reply-To: <373227.1744236488@sss.pgh.pa.us>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: <https://www.postgresql.org/message-id/4ea0957f-d599-47e5-ae9e-758ec6711e03%40technowledgy.de>
Precedence: bulk

Tom Lane:
> If you err in the other direction, you don't get slapped on the
> wrist that way.  We're willing to inline VOLATILE functions,
> for instance, whether or not the contained expression is volatile.
> Similarly for STRICT, and I think parallel safety as well.
> So my own habit when writing a SQL function that I wish to be
> inlined is to leave off all those markings.

According to [1], this only applies to inlining of scalar functions, but 
not to table functions, which *need* to be either STABLE or IMMUTABLE.

Just mentioning this for anyone taking this advice blindly and leaving 
all marks off, which might not always work as expected in the general case.

Best,

Wolfgang

[1]: https://wiki.postgresql.org/wiki/Inlining_of_SQL_functions