Re: BUG #17579: 15beta2: strange error when trying to use MERGE statement as a CTE

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Richard Guo <guofenglinux(at)gmail(dot)com>, borz_off(at)cs(dot)msu(dot)su, PostgreSQL mailing lists <pgsql-bugs(at)lists(dot)postgresql(dot)org>
Subject: Re: BUG #17579: 15beta2: strange error when trying to use MERGE statement as a CTE
Date: 2022-08-11 14:17:45
Message-ID: 20220811141745.pdor4vr2ktuauygk@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

In the meantime I noticed that DoCopy is subject to the same problem,
since it also uses PreparableStmt. I'll post a patch in a bit.

On 2022-Aug-11, Tom Lane wrote:

> The parse error location seems quite oddly chosen. Can you make
> it point at the MERGE instead? I think exprLocation(cte->ctequery)
> might work, but not sure.

I tried, but couldn't find the location anywhere. None of the nodes for
ModifyTable statements have location :-( We could add one now -- while
it's a bit late for 15, we're likely going to have a catversion bump due
to the JSON revert, so it might be fine.

--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"Cuando mañana llegue pelearemos segun lo que mañana exija" (Mowgli)

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2022-08-11 14:23:15 Re: BUG #17579: 15beta2: strange error when trying to use MERGE statement as a CTE
Previous Message Tom Lane 2022-08-11 13:53:33 Re: BUG #17206: the function array_cat(anyarray, anyarray) does not exist