public inbox for [email protected]
help / color / mirror / Atom feedFrom: Adrian Klaver <[email protected]>
To: Nico Williams <[email protected]>
To: Laurenz Albe <[email protected]>
Cc: Olleg Samoylov <[email protected]>
Cc: pgsql-generallists.postgresql.org <[email protected]>
Subject: Re: Interesting case of IMMUTABLE significantly hurting performance
Date: Wed, 9 Apr 2025 14:43:11 -0700
Message-ID: <[email protected]> (raw)
In-Reply-To: <Z/bk5E6pqIBWTF3j@ubby>
References: <[email protected]>
<[email protected]>
<Z/bk5E6pqIBWTF3j@ubby>
On 4/9/25 14:21, Nico Williams wrote:
> On Wed, Apr 09, 2025 at 10:50:00PM +0200, Laurenz Albe wrote:
>> The IMMUTABLE function cannot be inlined because to_char() is not IMMUTABLE.
>
> Q: Why would to_char() not be IMMUTABLE?
>
> A: Because it makes use of locales, and I guess the guc-timezone GUC,
> which could change if the expression is ultimately used in a PlPgSQL
> fragment, or if it's in a prepared statement. (I think.)
>
> That to_char is not immutable is not documented though. Though it's
> clear when looking at the docs for the `jsonb_.*_tz()` functions.
From here:
https://www.postgresql.org/docs/current/catalog-pg-proc.html
select proname, provolatile, prosrc from pg_proc where proname='to_char';
proname | provolatile | prosrc
---------+-------------+---------------------
to_char | s | timestamptz_to_char
to_char | s | numeric_to_char
to_char | s | int4_to_char
to_char | s | int8_to_char
to_char | s | float4_to_char
to_char | s | float8_to_char
to_char | s | interval_to_char
to_char | s | timestamp_to_char
Where 's' is:
"It is s for “stable” functions, whose results (for fixed inputs) do not
change within a scan."
>
> Nico
--
Adrian Klaver
[email protected]
view thread (11+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: Interesting case of IMMUTABLE significantly hurting performance
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox