Re: pgbench bug candidate: negative "initial connection time" - Mailing list pgsql-hackers

From Yugo NAGATA
Subject Re: pgbench bug candidate: negative "initial connection time"
Date
Msg-id 20210617125556.267049db17aaad74e9328965@sraoss.co.jp
Whole thread Raw
In response to Re: pgbench bug candidate: negative "initial connection time"  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: pgbench bug candidate: negative "initial connection time"  (Fabien COELHO <coelho@cri.ensmp.fr>)
List pgsql-hackers
On Wed, 16 Jun 2021 20:25:31 +0200 (CEST)
Fabien COELHO <coelho@cri.ensmp.fr> wrote:

> 
> Hello Yugo-san,
> 
> >> When connection break while the bench has already started,
> >> maybe it makes more sense to proceed,
> >
> > The result would be incomplete also in this case. However, the reason why
> > it is worth to proceed is that  such information is still useful for users,
> > or we don't want to waste the bench that has already started?
> 
> Hmmm. It depends on what the user is testing. If one is interested in 
> client resilience under errors, the bench should probably attempt a 
> reconnect. If one is interested in best performance when all is well,
> then clearly something is amiss and there is no point to go on.

Agreed. After --max-tries options is implemented on pgbench, we would be
able to add a new feature to allow users to choose if we try to reconnect
or not. However, we don't have it yet for now,  so we should just abort
the client and report the abortion at the end of the bench when a connection
or socket error occurs during the bench, as same the existing behaviour.

By the way, the issue of inital connection erros reported in this thread
will be fixed by the patch attached in my previous post (a major part are
written by you :-) ).  Is this acceptable for you?


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



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Improving isolationtester's data output
Next
From: Amit Kapila
Date:
Subject: Re: Decoding speculative insert with toast leaks memory