Re: BUG #18187: Unexpected error: "variable not found in subplan target lists" triggered by JOIN
Alexander Korotkov <aekorotkov@gmail.com>
From: Alexander Korotkov <aekorotkov@gmail.com>
To: Andrei Lepikhov <a.lepikhov@postgrespro.ru>
Cc: Richard Guo <guofenglinux@gmail.com>,
Alexander Korotkov <akorotkov@postgresql.org>, zuming.jiang@inf.ethz.ch, pgsql-bugs@lists.postgresql.org, PG Bug reporting form <noreply@postgresql.org>
Date: 2023-11-09T12:25:59Z
Lists: pgsql-bugs
On Thu, Nov 9, 2023 at 1:21 PM Andrei Lepikhov <a.lepikhov@postgrespro.ru> wrote: > On 9/11/2023 18:08, Richard Guo wrote: > > > > On Thu, Nov 9, 2023 at 12:51 PM Andrei Lepikhov > > <a.lepikhov@postgrespro.ru <mailto:a.lepikhov@postgrespro.ru>> wrote: > > > > ... Should we add a reference to the > > bug that triggered the issue as a comment to the test? > > > > > > Yeah, we can do that. I guess something like: > > > > -- Check that SJE removes references from PHVs correctly (bug #18187) > > > > Also, to be sure, > > maybe add column t4.code into the list of the coalesce parameters? > > > > > > I don't think it's necessary. It is t3 that SJE would remove, so it's > > sufficient to have t3's Vars in the PHV expression to trigger this > > error. > > Agree. I've considered the situations when something in the internal > planner logic changes, and relations could arrive in a different order. > In that case, we sort relids, and have determined behaviour. So, do we > wait for Alexander's glance? LGMT, pushed! ------ Regards, Alexander Korotkov
Commits
-
Fix a comment for remove_self_joins_recurse()
- 0a93f803f45f 17.0 landed
-
Don't constrain self-join removal due to PHVs
- b5fb6736ed3b 17.0 landed
-
Handle PlaceHolderVar case in replace_varno_walker
- 8a8ed916f73f 17.0 landed
-
Fix how SJE checks against PHVs
- b7f315c9d7d8 17.0 landed
-
citext: Allow tests to pass in OpenSSL FIPS mode
- 3c551ebede46 17.0 cited