public inbox for [email protected]
help / color / mirror / Atom feedFrom: Andrei Lepikhov <[email protected]>
To: Zsolt Parragi <[email protected]>
To: [email protected]
Subject: Re: Try a presorted outer path when referenced by an ORDER BY prefix
Date: Fri, 22 May 2026 07:51:04 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAN4CZFNXiuLztwqUCStR4VrCKH0WqHcsthP2CzkkFf5-k55=mA@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<CAN4CZFNXiuLztwqUCStR4VrCKH0WqHcsthP2CzkkFf5-k55=mA@mail.gmail.com>
On 13/05/2026 16:03, Zsolt Parragi wrote:
> + else if (jointype == JOIN_RIGHT)
> + ExecSetTupleBound(tuples_needed, innerPlanState(child_node));
>
> Is this branch reachable? If I follow the code correctly, it can't
> happen with create_nestloop_path.
RIGHT NL seems like nonsense to me. This code stays here just to remind me to
add a check in the next version, if the community is interested.
>
> pg_plan_advice also has test failures with the patch applied.
We already talked about this in the main thread for pg_plan_advice. Now, it is
unavoidable because pg_plan_advice limits optimisations that expand the planning
scope. Before, it was enough to use standard tools like create_seqscan_path and
add_path and provide a semantically correct plan. Now, every optimisation has to
meet pg_plan_advice's internal rules, which are much more restrictive.
I did not fix this test failure to highlight the unpleasant situation for
extension developers, but you can find the fix in the list [1].
[1]
https://www.postgresql.org/message-id/[email protected]...
--
regards, Andrei Lepikhov,
pgEdge
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]
Subject: Re: Try a presorted outer path when referenced by an ORDER BY prefix
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