Re: Bug in pgbench prepared statements - Mailing list pgsql-hackers

From Lev Kokotov
Subject Re: Bug in pgbench prepared statements
Date
Msg-id CAPwLVyKmWHA9CGEQyKEZuXrSoDGCqy+cNM+-uLAcwRrFysRpJQ@mail.gmail.com
Whole thread Raw
In response to Re: Bug in pgbench prepared statements  ("Tristan Partin" <tristan@neon.tech>)
Responses Re: Bug in pgbench prepared statements
List pgsql-hackers
> I see prepareCommand() is called one more time in
> prepareCommandsInPipeline(). Should you also check the return value
> there?

Yes, good catch. New patch attached.

> It may also be useful to throw this patch on the January commitfest if
> no one else comes along to review/commit it.

First time contributing, not familiar with the process here, but happy to learn.

Best,

Lev


On Thu, Nov 30, 2023 at 2:10 PM Tristan Partin <tristan@neon.tech> wrote:
On Wed Nov 29, 2023 at 7:38 PM CST, Lev Kokotov wrote:
> Patch attached, if there is any interest in fixing this small bug.

I see prepareCommand() is called one more time in
prepareCommandsInPipeline(). Should you also check the return value
there?

It may also be useful to throw this patch on the January commitfest if
no one else comes along to review/commit it. This first set of changes
looks good to me.

--
Tristan Partin
Neon (https://neon.tech)
Attachment

pgsql-hackers by date:

Previous
From: Andrei Lepikhov
Date:
Subject: Re: Report planning memory in EXPLAIN ANALYZE
Next
From: Nathan Bossart
Date:
Subject: Re: optimize atomic exchanges