From: | Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> |
---|---|
To: | kawamoto(at)sraoss(dot)co(dot)jp |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Typo in pgbench messages. |
Date: | 2022-02-24 03:15:55 |
Message-ID: | 20220224.121555.603427016970782829.t-ishii@sranhm.sra.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Hi,
>
> I found messages inserted a space before the "%" in pgbench.
> I think this is typo because there are no space before the "%" in other messages.
> What do you think?
I think you are right. In English there's should be no space between number and "%".
AFAIK other parts of PostgreSQL follow the rule.
> diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
> index f166a77e3a..4ebe5e6ea4 100644
> --- a/src/bin/pgbench/pgbench.c
> +++ b/src/bin/pgbench/pgbench.c
> @@ -5598,11 +5598,11 @@ printResults(StatsData *total,
> return;
>
> if (throttle_delay && latency_limit)
> - printf("number of transactions skipped: " INT64_FORMAT " (%.3f %%)\n",
> + printf("number of transactions skipped: " INT64_FORMAT " (%.3f%%)\n",
> total->skipped, 100.0 * total->skipped / total->cnt);
>
> if (latency_limit)
> - printf("number of transactions above the %.1f ms latency limit: " INT64_FORMAT "/" INT64_FORMAT " (%.3f %%)\n",
> + printf("number of transactions above the %.1f ms latency limit: " INT64_FORMAT "/" INT64_FORMAT " (%.3f%%)\n",
> latency_limit / 1000.0, latency_late, ntx,
> (ntx > 0) ? 100.0 * latency_late / ntx : 0.0);
Looks good to me.
Best reagards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2022-02-24 03:18:21 | Re: Design of pg_stat_subscription_workers vs pgstats |
Previous Message | Zheng Li | 2022-02-24 03:10:32 | Re: Support logical replication of DDLs |