Re: Typo in pgbench messages.

From: Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Michael Paquier <michael(at)paquier(dot)xyz>, Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp>, kawamoto(at)sraoss(dot)co(dot)jp, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Typo in pgbench messages.
Date: 2022-02-25 06:39:24
Message-ID: 20220225153924.a29bf2febbd0f7b5720f48f2@sraoss.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 24 Feb 2022 14:44:05 +0100
Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:

> > On 24 Feb 2022, at 13:58, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> wrote:
>
> >> One argument against a backpatch is that this would be disruptive with
> >> tools that parse and analyze the output generated by pgbench. Fabien,
> >> don't you have some tools and/or wrappers doing exactly that?
> >
> > Yep, I like to "| cut -d' ' -fx" and other "line.split(' ')" or whatever.
> >
> > I think that the break of typographical rules is intentional to allow such simplistic low-level stream handling through pipes or scripts. I'd prefer that the format is not changed. Maybe a comment could be added to explain the reason behind it.
>
> That doesn't sound like an overwhelmingly convincing argument to print some
> messages with 'X %' and others with 'X%'.

I agree with Daniel. If inserting a space in front of % was intentional for
handling the result in such tools, we should fix other places where 'X%' is
used in the pgbench output.

Regards,
Yugo Nagata

--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Julien Rouhaud 2022-02-25 07:03:52 Re: Report checkpoint progress with pg_stat_progress_checkpoint (was: Report checkpoint progress in server logs)
Previous Message Japin Li 2022-02-25 06:32:22 Re: [PATCH] add relation and block-level filtering to pg_waldump