public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andrew Dunstan <[email protected]>
To: postgresql_contributors <[email protected]>
To: [email protected] <[email protected]>
Cc: Kiran Suresh Kulkarni <[email protected]>
Cc: Hari Prasad B <[email protected]>
Subject: Re: Guidance Needed for Testing PostgreSQL Patch (CF-5044)
Date: Wed, 4 Dec 2024 09:08:53 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <TYZPR03MB816867F0BAAE37EF3445B89DE1372@TYZPR03MB8168.apcprd03.prod.outlook.com>
References: <TYZPR03MB816867F0BAAE37EF3445B89DE1372@TYZPR03MB8168.apcprd03.prod.outlook.com>


On 2024-12-04 We 3:07 AM, postgresql_contributors wrote:
> Hi pg_hackers,
>
> I hope you are doing well. I am working on reviewing PostgreSQL 
> community tickets and started with the patch cf/5044.
> This patch is in the branch cf/5044, and I pulled the code to test it.
> Here’s what I have done so far:
>
>  *
>     Pulled the code from the branch cf/5044.
>  *
>     Compiled the code.
>  *
>     Set up the database as required.
>
> Tested the query for this patch mentioned in the community chat 
> (https://commitfest.postgresql.org/50/5044/ 
> <https://commitfest.postgresql.org/50/5044/;) .
> However, I didn’t get the expected output mentioned in the ticket 
> (below mentioned example).
>
> CREATE TABLE foo3(id serial PRIMARY key, txt text);
>
> INSERT INTO foo3 (txt) VALUES ('aaa'),('bbb');
> DO $$
> DECLARE
>     l_cnt int;
> BEGIN
>     l_cnt := 1
>     DELETE FROM foo3 WHERE id=1;
> END; $$
>
> -- without reaction - just don't work
>
> (2024-06-16 16:05:55) postgres=# set plpgsql.extra_errors to
> 'strict_expr_check';
> SET
> (2024-06-16 16:06:43) postgres=# DO $$
>
> DECLARE
>     l_cnt int;
> BEGIN
>     l_cnt := 1
>     DELETE FROM foo3 WHERE id=1;
> END; $$;
> ERROR:  syntax error at or near "DELETE"
> LINE 11:     DELETE FROM foo3 WHERE id=1;
>              ^
> Please provide guidance on this patch.
>

Well, it's told you what the problem is - there's a syntax error. And 
it's pretty glaring: the line


     l_cnt := 1


is missing a trailing semicolon. So I suggest you add that and then keep 
going.


cheers


andrew


--
Andrew Dunstan
EDB:https://www.enterprisedb.com

view thread (2+ messages)

reply

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: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Guidance Needed for Testing PostgreSQL Patch (CF-5044)
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox