Re: pg_plan_advice

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Lukas Fittl <lukas(at)fittl(dot)com>
Cc: Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Dian Fay <di(at)nmfay(dot)com>, Matheus Alcantara <matheusssilv97(at)gmail(dot)com>, Jakub Wartak <jakub(dot)wartak(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz>
Subject: Re: pg_plan_advice
Date: 2026-01-08 16:37:51
Message-ID: CA+TgmoZCYd1iEAZAYn0ie02mQ=Jc_v=8JRFdJRBDkdkOGLhBTg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jan 8, 2026 at 11:31 AM Lukas Fittl <lukas(at)fittl(dot)com> wrote:
> Yeah, I would agree with you that is inconsistent with the flag's name
> - but on the flip side, its difficult for the caller to conditionally
> set the flag (which you'd have to do to avoid a "Disabled" showing in
> the plan), since we're setting it on the RelOptInfo (do we know if the
> scan is a sample scan at that point?).

How about checking rte->tablesample, as set_rel_pathlist does?

--
Robert Haas
EDB: http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrey Borodin 2026-01-08 16:49:31 Re: GIN pageinspect support for entry tree and posting tree
Previous Message Lukas Fittl 2026-01-08 16:30:42 Re: pg_plan_advice