Re: Print latency percentiles in pgbench

From: Hữu Đôn Hoàng <hoanghuudon02hp(at)gmail(dot)com>
To: Roman Khapov <rkhapov(at)yandex-team(dot)ru>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Print latency percentiles in pgbench
Date: 2026-11-04 11:49:38
Message-ID: CALSjoXhLfhJ8aFKSRaNHNOKTct62Goyy+X-b0b==y2b=yh7VyA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Roman,

> POC-patch attached, any feedback would be appreciated!

Thanks for working on this. I've started with the documentation part
of the patch, on master at 30faf13889b.

The doc changes re-indent some existing text without changing it:

- the <varlistentry> and <term> lines of --log-prefix,
- the "when neither --progress, --rate, nor --latency-limit" lines,
- the whole paragraph on retries ("The main report contains the
number of failed transactions...").

The new --latency-percentiles varlistentry is also indented one level
deeper than the entries around it, so --log-prefix now has its
<varlistentry> and <term> at a different depth from its <listitem>.

Because of this, the doc diff is +57/-12, while git diff -w shows
+47/-2. Could you keep the existing indentation and indent the new
entry like its neighbours, so that only the real changes remain?

Regards,
Don Hoang

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Raja Sai pranav 2026-11-04 11:56:17 Re: Re: [PATCH] Fix use-after-free after failed pg_checksum_init
Previous Message Zsolt Parragi 2026-11-04 11:27:48 Re: Custom oauth validator options