| From: | Richard Guo <guofenglinux(at)gmail(dot)com> |
|---|---|
| To: | Andrei Lepikhov <a(dot)lepikhov(at)postgrespro(dot)ru> |
| Cc: | Alexander Korotkov <aekorotkov(at)gmail(dot)com>, zuming(dot)jiang(at)inf(dot)ethz(dot)ch, pgsql-bugs(at)lists(dot)postgresql(dot)org, PG Bug reporting form <noreply(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Re: BUG #18187: Unexpected error: "variable not found in subplan target lists" triggered by JOIN |
| Date: | 2023-11-22 08:59:21 |
| Message-ID: | CAMbWs4_XreeMgvDMPDGPWYdz5fh3X1v2Qe_XRaN3d1g72URi2A@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
On Wed, Nov 22, 2023 at 10:32 AM Andrei Lepikhov <a(dot)lepikhov(at)postgrespro(dot)ru>
wrote:
> Your patch is ok.
Thanks for looking into this patch.
> I think the comment, 'PHVs should not impose ...' looks redundant. It
> may be enough to have it in the commit comment.
Hmm, I'm not sure. I'm thinking that when people read this part of the
code, they might be confused as to why we don't consider PHVs. And this
comment will give them some clues. This is just like how we explain why
we need not worry about LATERAL nor PHVs in is_simple_values().
Thanks
Richard
| From | Date | Subject | |
|---|---|---|---|
| Next Message | PG Bug reporting form | 2023-11-22 12:21:25 | BUG #18210: libpq: PQputCopyData sometimes fails in non-blocking mode over GSSAPI encrypted connection |
| Previous Message | PG Bug reporting form | 2023-11-22 03:39:31 | BUG #18209: New connection is waiting for ProcArrayLock lock when execute a stored procedure concurrently |