From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrei Lepikhov <lepihov@gmail.com>
Cc: Richard Guo <guofenglinux@gmail.com>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #18576: Using EXPLAIN (VERBOSE) in information_schema.element_types returns ERROR: failed to find plan for
Date: Mon, 19 Aug 2024 13:26:50 -0400
Message-ID: <1390980.1724088410@sss.pgh.pa.us> (raw)
In-Reply-To: <5e129a8e-5b8d-4a61-9776-59be99a2c14f@gmail.com>
References: <18576-9feac34e132fea9e@postgresql.org>
<CAMbWs49gOeyotM7riJ-q_NgLgfA6-3z7DffnR71kSyaHTkh2ig@mail.gmail.com>
<2336091.1723137115@sss.pgh.pa.us>
<2493069.1723149304@sss.pgh.pa.us>
<CAMbWs4_Au2i+p2MbZ8nNB_QQXrFy65CXNeJL11j2WJbNW18fHQ@mail.gmail.com>
<2548861.1723174234@sss.pgh.pa.us>
<2799439.1723217127@sss.pgh.pa.us>
<332f0e30-f90a-4e47-b668-b53f649c244f@gmail.com>
<1385302.1724085401@sss.pgh.pa.us>
<5e129a8e-5b8d-4a61-9776-59be99a2c14f@gmail.com>
Andrei Lepikhov <lepihov@gmail.com> writes:
> On 19/8/2024 18:36, Tom Lane wrote:
>> This seems like it's making assumptions it shouldn't about what
>> CustomScan does. If there's an argument for doing this, it should
>> be added to the adjacent comments.
> Hm, I got into this problem many times using CustomScan node. Do you
> have some objections to not allow CustomScan node have a RECORD Var in
> the target list?
In the case of a childless Result, we can suppose that the reason why
we're here is that a provably-empty subquery got optimized away.
If the Var were actually evaluated at runtime, it would fail, so that
had better be the case. (I thought about extending these new Asserts
to check that the Result has a constant-false resconstantqual, but
decided that was overkill.)
It's not clear to me what the equivalent argument is for allowing
CustomScan. I don't say that there isn't one. I do say that a patch
like this should make that argument, in the same comment block that
explains why we're doing this for Result.
The main reason I'm being sticky about this is that if we need to
allow CustomScan, then it seems likely that we also need to allow
ForeignScan, and maybe some other things, and then I start to
wonder if we should have any assertion at all about the child
plan type. So I want to actually understand what is the scenario
in which this will happen.
regards, tom lane
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: pgsql-bugs@postgresql.org
Cc: tgl@sss.pgh.pa.us, lepihov@gmail.com, guofenglinux@gmail.com, pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #18576: Using EXPLAIN (VERBOSE) in information_schema.element_types returns ERROR: failed to find plan for
In-Reply-To: <1390980.1724088410@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox