Re: CASE control block broken by a single line comment - Mailing list pgsql-hackers

From Tom Lane
Subject Re: CASE control block broken by a single line comment
Date
Msg-id 2730343.1712464392@sss.pgh.pa.us
Whole thread Raw
In response to Re: CASE control block broken by a single line comment  (Erik Wienhold <ewie@ewie.name>)
Responses Re: CASE control block broken by a single line comment
List pgsql-hackers
Erik Wienhold <ewie@ewie.name> writes:
> On 2024-04-06 20:14 +0200, Michal Bartak wrote:
>> The issue described bellow exists in postgresql ver 16.2 (found in some
>> previous major versions)

> Can confirm also on master.

I'm sure it's been there a while :-(

> I'm surprised that the comment is not skipped by the scanner at this
> point.  Maybe because the parser just reads the raw expression between
> WHEN and THEN with plpgsql_append_source_text via read_sql_construct.

> How about the attached patch?  It's a workaround by simply adding a line
> feed character between the raw expression and the closing parenthesis.

I don't have time to look into this on this deadline weekend, but
what's bothering me about this report is the worry that we've made
the same mistake elsewhere, or will do so in future.  I suspect
it'd be much more robust if we could remove the comment from the
expr->query string.  No idea how hard that is.

            regards, tom lane



pgsql-hackers by date:

Previous
From: jian he
Date:
Subject: Re: remaining sql/json patches
Next
From: Andres Freund
Date:
Subject: Re: Improve heapgetpage() performance, overhead from serializable