Re: pgbench: allow to exit immediately when any client is aborted - Mailing list pgsql-hackers

From Yugo NAGATA
Subject Re: pgbench: allow to exit immediately when any client is aborted
Date
Msg-id 20230809135921.87780d9983d3081994901ca7@sraoss.co.jp
Whole thread Raw
In response to Re: pgbench: allow to exit immediately when any client is aborted  (Yugo NAGATA <nagata@sraoss.co.jp>)
Responses Re: pgbench: allow to exit immediately when any client is aborted
List pgsql-hackers
On Wed, 9 Aug 2023 10:46:38 +0900
Yugo NAGATA <nagata@sraoss.co.jp> wrote:

> On Wed, 9 Aug 2023 02:15:01 +0200 (CEST)
> Fabien COELHO <coelho@cri.ensmp.fr> wrote:
> 
> > 
> > Hello Yugo-san,
> > 
> > > There are cases where "goto done" is used where some error like
> > > "invalid socket: ..." happens. I would like to make pgbench exit in
> > > such cases, too, so I chose to handle errors below the "done:" label.
> > > However, we can change it to call "exit" instead of "goo done" at each
> > > place. Do you think this is better?
> > 
> > Good point.
> > 
> > Now I understand the "!= FINISHED", because indeed in these cases the done 
> > is reached with unfinished but not necessarily ABORTED clients, and the 
> > comment was somehow misleading.
> > 
> > On reflection, there should be only one exit() call, thus I'd say to keep 
> > the "goto done" as you did, but to move the checking loop *before* the 
> > disconnect_all, and the overall section comment could be something like 
> > "possibly abort if any client is not finished, meaning some error 
> > occured", which is consistent with the "!= FINISHED" condition.
> 
> Thank you for your suggestion.
> I'll fix as above and submit a updated patch soon.

I attached the updated patch v3 including changes above, a test,
and fix of the typo you pointed out.

Regards,
Yugo Nagata

> 
> Regards,
> Yugo Nagata
> 
> -- 
> Yugo NAGATA <nagata@sraoss.co.jp>
> 
> 


-- 
Yugo NAGATA <nagata@sraoss.co.jp>

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: pg_upgrade fails with in-place tablespace
Next
From: Noah Misch
Date:
Subject: Re: 2023-08-10 release announcement draft