Re: pgbench bug candidate: negative "initial connection time"

Поиск
Список
Период
Сортировка
От Yugo NAGATA
Тема Re: pgbench bug candidate: negative "initial connection time"
Дата
Msg-id 20210617125556.267049db17aaad74e9328965@sraoss.co.jp
обсуждение исходный текст
Ответ на Re: pgbench bug candidate: negative "initial connection time"  (Fabien COELHO <coelho@cri.ensmp.fr>)
Ответы Re: pgbench bug candidate: negative "initial connection time"  (Fabien COELHO <coelho@cri.ensmp.fr>)
Список 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 по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Improving isolationtester's data output
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: Decoding speculative insert with toast leaks memory