Re: [PATCH] add long options to pgbench (submission 1)

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, fabriziomello(at)gmail(dot)com, PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] add long options to pgbench (submission 1)
Date: 2013-06-27 13:02:39
Message-ID: CA+TgmoYdLa-_OF4W=a_dkhJ7F_UxB7V-J7-JOaWvs8VzM1dbiw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jun 25, 2013 at 3:09 PM, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> wrote:
>> I think --quiet-log should be spelled --quiet.
>
> ISTM that --quiet usually means "not verbose on stdout", so I added log
> because this was specific to the log output, and that there may be a need
> for a --quiet option for stdout at some time.

The output that is quieted by -q is not the log output produced by
--log; it's the regular progress output on stdout/stderr.

So I changed that, and committed this, with some further cosmetic
changes. I made the formatting of the help message more like psql's
help message, including adjusting pgbench to start the description of
each option in the same column that psql does. This got rid of a lot
of line breaks and IMHO makes the output of pgbench --help quite a bit
more readable. I made stylistic adjustments to the documentation
portion of the patch as well, again to match the markup used for psql.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Markus Wanner 2013-06-27 13:05:58 Re: Hash partitioning.
Previous Message Andres Freund 2013-06-27 13:01:46 Re: Move unused buffers to freelist