Re: pgbench - compute & show latency consistently

From: Kuntal Ghosh <kuntalghosh(dot)2007(at)gmail(dot)com>
To: coelho(at)cri(dot)ensmp(dot)fr
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgbench - compute & show latency consistently
Date: 2016-09-21 10:07:26
Message-ID: CAGz5QCJjBxLgqt9=uC3gRJUnTGh6BZsMivs7iyPLw0LD_TURsw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thus, July 7,2016 at 08:39 PM, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> wrote:
> Also there is still the bug under -t which displays a 0 latency.
Your patch clearly fixed the issue.

> The attached patch still fixes that and make it consistent the other way
> around, i.e. by using "=" for latency. I switched to use ":" for weight
> which is an input parameter. I let ":" when there is a long sentence to
> describe the figure displayed, more on aesthetic grounds.
In the above context, I suggest few other changes.

Present output-> progress: 1.0 s, 1221.0 tps, lat 0.816 ms stddev 0.272
Suggestion-> progress: 1.0 s, 1221.0 tps, lat avg 0.816 ms stddev 0.272 ms

Present output->
SQL script 1: so.sql
- weight = 1 (targets 50.0% of total)
- 10010 transactions (50.1% of total, tps = 100.101872)
- latency average = 1.878 ms
- latency stddev = 3.614 ms
Suggestion->
SQL script 1: so.sql
- weight = 1 (targets 50.0% of total)
- 10010 transactions (50.1% of total)
- tps = 100.101872
- latency average = 1.878 ms
- latency stddev = 3.614 ms

Apart from that, pgbench.sgml should be updated to reflect latency
average in the output.

--
Thanks & Regards,
Kuntal Ghosh
EnterpriseDB: http://www.enterprisedb.com

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Heikki Linnakangas 2016-09-21 10:30:27 Re: pgbench - compute & show latency consistently
Previous Message Petr Jelinek 2016-09-21 09:54:16 Re: Quorum commit for multiple synchronous replication.