MIME-Version: 1.0
References: 
 <CA+TgmoZ-Jh1T6QyWoCODMVQdhTUPYkaZjWztzP1En4=ZHoKPzw@mail.gmail.com>
 <CAKZiRmxtJAFG7e1+Vs9B8ngON=AOzJbuws+1ZeH4LsbJh5AzoQ@mail.gmail.com>
 <CA+TgmoY9Ne_Sh10u6LSPc3wvOQPLp3kF9nBp3nqJEG2JGF2QiA@mail.gmail.com>
 <CA+Tgmoa57S6mP=aTOXH2-gDAL4TMO1WbGgrHSg0s6J4zUH=04g@mail.gmail.com>
 <DEB1SAGVDX7S.JMV96X4R0RCS@gmail.com>
 <CA+Tgmoaf__2B0BUL+vrg28P+3buX=Ti-kybqkHiLTtFrrCfzuA@mail.gmail.com>
 <CA+TgmoYpcLNOuypOTdgCSLW7FuA=t6BtB3meTARHX2-Dj_81xQ@mail.gmail.com>
 <DEFNP3WX7BLX.1X7DS1P9X6JIF@nmfay.com>
 <CA+TgmoZjv9OyFu1Gkt78w0vWEti8S33w8trYHmErf-GMmGSi=w@mail.gmail.com>
 <DELPC4B56Y5A.3OBZ3WCH3T4VX@nmfay.com>
 <CA+TgmoaOSBQD9Ux4eG40w723ZN=c0J7p-+oX4+J8urUeyLMo5w@mail.gmail.com>
 <CAOYmi+=g+MMoOpWkk2weXWKJcKH0eKey8gKHHdH0dF4Tiawrhw@mail.gmail.com>
 <CA+TgmobwaT=PXPDDrgDup+jA8KHBbkxigtziD-zNzAKKkQYVgQ@mail.gmail.com>
 <CAOYmi+mOmEW=amDRQMfw6-Fb3ZmDEQFaJzwk8Bc8W8DzaP85XQ@mail.gmail.com>
 <CA+TgmoaX2AMW4cdFM3OngBJxmxpkdmzF33R7-CWhvRLfucbFMg@mail.gmail.com>
 <CAOYmi+k4AyWCQHK=XVF99KVDuFkqxcADao61OWGLxu0nRYMONQ@mail.gmail.com>
 <CA+TgmoZ0x3ym_oueXRWzbM_=6ucKoPZVGj3rRMLBDC_FnetXDw@mail.gmail.com>
 <CAP53Pkycc=7N2bLzVT3x+qE1JamvRZWev5tFjdLJ1+-AV3Di+Q@mail.gmail.com>
 <CA+TgmoaKhuD91RnazbRyGkmP7--JdNq8oNDC3UcgTZSWbMxC7w@mail.gmail.com>
 <CAP53Pkw5-wMEeDJXFmqo_RTyL_spzCXb7HHKrbSnQqokVoZcNQ@mail.gmail.com>
 <CA+Tgmob-69bzbdi3U_QtebqAf6u1y8js=5=oNK639csVe1VbhA@mail.gmail.com>
 <CA+TgmoaZMOikxK=LqS+Jn+835h9S139JLGk-3LyETVXw5W5j=w@mail.gmail.com>
 <1136161.1769654478@sss.pgh.pa.us>
 <CAP53PkwZ1ZTMARKg6iEfAw9qzBhkjBitj-9gr_Jvy7k2AwGgWA@mail.gmail.com>
 <CAMbWs4--NuEUFE_xTo991TRXaZryE29jarJPDnVxoaQOYdt7tA@mail.gmail.com>
 <CA+TgmobzR+XMGbRosVPbjHbSo4+cgJn=qZK6w05aF1sbj=C+9Q@mail.gmail.com>
 <CA+TgmoawzvCoZAwFS85tE5+c8vBkqgcS8ZstQ_ohjXQ9wGT9sw@mail.gmail.com>
 <CA+TgmoYS4ZCVAF2jTce=bMP0Oq_db_srocR4cZyO0OBp9oUoGg@mail.gmail.com>
 <CAK98qZ2RzbgCHrSg4zLkvpzyBam_X6te-KF8w1+_vON9BAVMEw@mail.gmail.com>
 <CA+TgmoaCdsuvNn6T6SfQ_0YD2Hh2+hgTXh9fTGHQhPg1zvy2rQ@mail.gmail.com>
 <CAK98qZ1zWzRB0ABG7ULzTeWKRR5C7-FxLqM-6v8wQDiFM+DNAg@mail.gmail.com>
 <CA+Tgmob7ozJAs5SU6bD2RfAt4w_AmsMGz-aaVA6WeLXHkBypOg@mail.gmail.com>
 <CAK98qZ1J42RoAsHnYWGPPmHziZmzmqE=Lp_O6WJ-9aKK2qjikA@mail.gmail.com>
 <CA+TgmoYjcBA6dw3nwiyfDzPXTCrxTZPXDMrc2TrDJcL1cPK6iA@mail.gmail.com>
 <CA+TgmoYru-vxoTKfwjQby30r2OkTXfb18Km_=VLs6qk8Akr0-g@mail.gmail.com>
In-Reply-To: 
 <CA+TgmoYru-vxoTKfwjQby30r2OkTXfb18Km_=VLs6qk8Akr0-g@mail.gmail.com>
From: Jakub Wartak <jakub.wartak@enterprisedb.com>
Date: Tue, 3 Mar 2026 12:42:13 +0100
Message-ID: 
 <CAKZiRmx=ijCZFpAYMb1z0=9u0iixqD6cBKPBx+WLFFKqOW8R=w@mail.gmail.com>
Subject: Re: pg_plan_advice (now with transparent SQL plan performance
 overrides - pg_stash_advice)
To: Robert Haas <robertmhaas@gmail.com>
Cc: Alexandra Wang <alexandra.wang.oss@gmail.com>,
 Richard Guo <guofenglinux@gmail.com>,
	Lukas Fittl <lukas@fittl.com>, Tom Lane <tgl@sss.pgh.pa.us>,
	Jacob Champion <jacob.champion@enterprisedb.com>, Dian Fay <di@nmfay.com>,
	Matheus Alcantara <matheusssilv97@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAKZiRmx%3DijCZFpAYMb1z0%3D9u0iixqD6cBKPBx%2BWLFFKqOW8R%3Dw%40mail.gmail.com>
Precedence: bulk

On Fri, Feb 27, 2026 at 11:46=E2=80=AFPM Robert Haas <robertmhaas@gmail.com=
> wrote:
>
> On Thu, Feb 26, 2026 at 8:55=E2=80=AFAM Robert Haas <robertmhaas@gmail.co=
m> wrote:
> Here's v18. [..]
[..]
> Second, I also added a third contrib module called pg_stash_advice.
> This uses the same hook that I previously added for test_plan_advice,
> but unlike that module, this one's not just a test. It lets you set up
> an "advice stash" which is basically a query_id->advice_string hash
> table. If you then set pg_stash_advice.stash_name to the name of your
> advice stash, it will do a lookup into that hash table every time a
> query is planned and, if the query ID is found, it will do the
> planning with the corresponding advice string. What I think is
> particularly cool about this is that it shows that you can really use
> that hook to apply advice on the fly in any way you want. I suspect
> that query ID matching will be suitable for a lot of use cases, but
> you could have a similar module that matches on query text or does
> anything else that you want as long as an advice string pops out at
> the end. It shows that the core pg_plan_advice infrastructure is
> pluggable. So this is both something that I think a lot of people will
> find useful all on its own, and also a design pattern that people can
> copy and adapt.

Hi Robert, I'm glad that you posted this. I've thought this is so important=
 that
I've sligthly added more descriptive subject as this is NEW and easily to m=
iss
out as it was burried in the second paragraph and most folks probably would=
 miss
that this $thread is now includes code for transparent SQL plan
overrides (most known
as baselines or SQL plan management) - it has many names, but I havent come
across "stash" as one of them, so people could miss it too easily.

1. First thought is that I found it quite surprising, we now have 3 modules
and it's might cause confusion and lack of consistency:
- 3 can to be in shared_preload_libraries (pg_stash_advice,
pg_plan_advice, pg_collecti_advice)
- 2 others can use CREATE EXTENSION (pg_stash_advice, pg_collect_advice), b=
ut
  "create extension pg_plan_advice;" fails
so maybe they all should behave the same as people (including me) won't rea=
d
the docs and just blindly add it here and there and issue CREATE EXTENSION,
but it's going to be hard to remember for which ones (?) So we need more
consistency?

2. Should pgca always duplicate entries like that? (each call gets new entr=
y?)
Shouldn't it just update collection_time for identical calls of
userid/dbid/queryid?

postgres=3D# select * from pg_get_collected_shared_advice();
 id | userid | dbid |       queryid       |        collection_time
   |                      query                      |
 advice
----+--------+------+---------------------+-------------------------------+=
-------------------------------------------------+-------------------------=
------------------
  0 |     10 |    5 | 1069089066624131207 | 2026-03-03
09:47:40.322294+01 | select * from pg_get_collected_shared_advice(); |
NO_GATHER(pg_get_collected_shared_advice)
(1 row)

postgres=3D# select * from pg_get_collected_shared_advice();
 id | userid | dbid |       queryid       |        collection_time
   |                      query                      |
 advice
----+--------+------+---------------------+-------------------------------+=
-------------------------------------------------+-------------------------=
------------------
  0 |     10 |    5 | 1069089066624131207 | 2026-03-03
09:47:40.322294+01 | select * from pg_get_collected_shared_advice(); |
NO_GATHER(pg_get_collected_shared_advice)
  1 |     10 |    5 | 1069089066624131207 | 2026-03-03
09:47:41.983973+01 | select * from pg_get_collected_shared_advice(); |
NO_GATHER(pg_get_collected_shared_advice)
(2 rows)

It floods like that for everything, most visible with couple of independent
starts of pgbench -M prepared. Maybe I'm wrong , but I don't think it worke=
d
like that earlier before refactor?

3. The good news is that I have managed to finally overrride SQL plans
completley transparently using set of those modules for online pgbench runs=
.
However I found couple of issues.

3a. Because query_id each time will be different for every query even
in standard
pgbench mode, I've managed to achieve it only using prepared statements.
Realistically we'll need some way of modular matching not just on query_id =
OR
query_id should be changed how it being calculcated or maybe by some other =
means
(argument is: not all users/apps use prepared statements):
- at least some minimal query jumbling, part of me belives that e.g. code f=
rom
  62d712ecfd940 / pgss's generate_normalized_query() should be more reusabl=
e
  across other extensions, and then why not just use it here?
- maybe it should: ignore uppercase vs lowercase, removal of extra whitespa=
ces
- maybe it should: ignore schema names (in multi-tenant shops)
- maybe it should: remove SQL comments (/*+  xxx */) or newlines (as there =
are
  client-side libraries that annotate SQL queries by putting comments to lo=
cate
  e.g. app source code location/origin)
- maybe we even should have some regexp

I'm not buying argument that it will make something slower, because :
a) this is on-demand loaded module for those who want it
b) it exists purely to avoid way bigger performance problems in the first p=
lace

3b. When performing manual testing using PREPARE s1 / EXECUTE s1, the SQL p=
lans
are effective out of the box. Below I'm intentionally downgrading runtime
performance and that works:

postgres=3D# PREPARE p1 (int) AS UPDATE pgbench_accounts SET abalance =3D
abalance + $1 WHERE aid =3D $2;
PREPARE
postgres=3D# \timing on
Timing is on.
postgres=3D# EXECUTE p1(42, 42);
UPDATE 1
Time: 8.241 ms

-- making it slow, queryid from explain (verbose), same session:
postgres=3D# select pg_set_stashed_advice('abc456',
-9041336337128051785, 'SEQ_SCAN(pgbench_accounts)');
 pg_set_stashed_advice
-----------------------

(1 row)

Time: 0.995 ms
postgres=3D# show pg_stash_advice.stash_name ;
 pg_stash_advice.stash_name
----------------------------
 abc456
(1 row)

Time: 0.366 ms
-- OK, that's effective (up from 8ms)
postgres=3D# EXECUTE p1(42, 42);
UPDATE 1
Time: 448.415 ms

3c. However for pgbench -M prepared, such online plan alterations are
strangley not effective. Even creating new stash under different name, sett=
ing
GUC, and pg_reloading_conf is not effective too for hundreths of seconds of
pgbench.

3d. ... however restarting restarting pgbench helps and the session
changes plan (and thus
performance characteristics). When using manual way of reproducing
this more like:

sess1=3D# PREPARE p1 (int) AS UPDATE pgbench_accounts SET abalance =3D
abalance + $1 WHERE aid =3D $2;
PREPARE
sess1=3D# EXECUTE p1(42, 42);
UPDATE 1
sess1=3D# \timing on
Timing is on.
sess1=3D# EXECUTE p1(42, 42);
UPDATE 1
Time: 8.823 ms

-- now from 2nd session
sess2=3D# select pg_set_stashed_advice('abc456', -9041336337128051785,
'SEQ_SCAN(pgbench_accounts)');

-- .. it was NOT effective, and still fast, but it should be slow:
sess1=3D# EXECUTE p1(42, 42);
UPDATE 1
Time: 8.781 ms
postgres=3D#

However doing it from sess2 sometimes, makes it often faster effective:
postgres=3D# select pg_create_advice_stash('xyz123');
postgres=3D# select pg_set_stashed_advice('xyz123',
-9041336337128051785, 'SEQ_SCAN(pgbench_accounts)');
postgres=3D# alter system set pg_stash_advice.stash_name TO 'xyz123';
postgres=3D# select pg_reload_conf();

but apparently that was still not solving the pgbench case problem.
So it doesn't seem to be deterministic when the new plan is applied (?)

3e. As this was pretty concerning, I've repeated the pgbench -M prepared
excercise and I've figured out that I could achieve effect that I wanted (
boucing between fast <-> slow immediatley) by injecting call via gdb on tha=
t
backend to InvalidateSystemCaches(). Kind of brute-force, but only
then it worked
instantly the pgbench backend started using new "stash" immediatley). Quest=
ion
should or should not it immediatley effective and have you got any idea why
there is such difference in behaviour between pgbench and psql? Should we
investigate it further? I think, we should?

4. The familiy of pg_*stash*() functions could return some ::bool result in=
stead
of void. Like true? (it's usage "feeling" that leaves one wondering if
the command
was effective or not, e.g. to get consistency with let's say pg_reload_conf=
())

5. QQ: will pg_stash_advice persist the stashes one day?

6. Any idea for better name than 'stash' ? :) It's some new term that is fo=
r
sure not wildly recognized. Some other used name across industry: plan
stability,
advice freeze, plan freeze, plan force, baselines, plan management, force p=
lan,
force paths, SQL plan optimization profiles, query store (MSSQL).

7. I saw you have written that in docs to be careful about memory use, but
wouldn't be it safer if that maximum memory for pgca (when collecting in sh=
ared
with almost infinite limite) would be still subject to like let's say 5% of=
 s_b
(or any other number here)?

8. I'm wondering if we should apply standard PostgreSQL case insensitivity
rule (e.g. like for relations) for those stashes? On one front we ignore ca=
se
sensitivty for objects, one another this is GUC so perhaps it is OK (I
feel it is
ok, but I wanted to ask).

9. If IsQueryIdEnabled() is false (even after trying out to use 'auto')
shouldn't this module raise warning when pg_stash_advice.stash_name !=3D NU=
LL?

> [..altered sequence],
[..]
> First I realized that it might be confusing to have the collector
> interface as part of pg_plan_advice, because for most of what
> pg_plan_advice does, you didn't need the extension, but for that part,
> you did. So, I broke that part out into its own extension, now called
> pg_collect_advice, [..]

10. I'm was here mainly for v18-0007 (pg_stash_advice), but this still look=
s
like some small bug to me (minmax matching in v18-0003):

create table t1  as select * from generate_series(1, 100000) as id;
create unique index t1_pk on t1 (id);
analyze t1;

-- OK "matched"
postgres=3D# set pg_plan_advice.advice to 'INDEX_ONLY_SCAN(t1@minmax_1
public.t1_pk)';
SET
postgres=3D# explain (plan_advice, costs off) select max(id) from t1;
                        QUERY PLAN
-----------------------------------------------------------
 Result
   Replaces: MinMaxAggregate
   InitPlan minmax_1
     ->  Limit
           ->  Index Only Scan Backward using t1_pk on t1
                 Index Cond: (id IS NOT NULL)
 Supplied Plan Advice:
   INDEX_ONLY_SCAN(t1@minmax_1 public.t1_pk) /* matched */
 Generated Plan Advice:
   INDEX_ONLY_SCAN(t1@minmax_1 public.t1_pk)
   NO_GATHER(t1@minmax_1)
(11 rows)

Manual SET, wont work and it is failed (which is OK)

postgres=3D# set pg_plan_advice.advice to 'SEQ_SCAN(t1)';
SET
postgres=3D# explain (plan_advice, costs off) select max(id) from t1;
                        QUERY PLAN
----------------------------------------------------------
 Result
   Replaces: MinMaxAggregate
   InitPlan minmax_1
     ->  Limit
           ->  Index Only Scan Backward using t1_pk on t1
                 Index Cond: (id IS NOT NULL)
 Supplied Plan Advice:
   SEQ_SCAN(t1) /* matched, failed */
 Generated Plan Advice:
   INDEX_ONLY_SCAN(t1@minmax_1 public.t1_pk)
   NO_GATHER(t1@minmax_1)

However with below SEQ_SCAN is applied/matched, but marked as failed
(so bug?):

postgres=3D# set pg_plan_advice.advice to 'SEQ_SCAN(t1@minmax_1)';
SET
postgres=3D# explain (plan_advice, costs off) select max(id) from t1;
                  QUERY PLAN
-----------------------------------------------
 Aggregate
   ->  Seq Scan on t1
 Supplied Plan Advice:
   SEQ_SCAN(t1@minmax_1) /* matched, failed */
 Generated Plan Advice:
   SEQ_SCAN(t1)
   NO_GATHER(t1)

-J.