Re: Better help output for pgbench -I init_steps

From: Tristen Raab <tristen(dot)raab(at)highgo(dot)ca>
To: pgsql-hackers(at)lists(dot)postgresql(dot)org
Cc: Gurjeet Singh <gurjeet(at)singh(dot)im>
Subject: Re: Better help output for pgbench -I init_steps
Date: 2023-09-22 21:01:34
Message-ID: 169541649451.2315936.3667259122357722951.pgcf@coridan.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: tested, passed
Spec compliant: not tested
Documentation: not tested

Hello,

I've reviewed all 4 of your patches, each one applies and builds correctly.

> I think I prefer variant 2. Currently, we only have 8 steps, so it might
> be overkill to separate them out into a different option.

+1 to this from Peter. Variant 2 is nicely formatted with lots of information which I feel better solves the problem this patch is trying to address.
Both versions 3 and 4 are a bit too jumbled for my liking without adding anything significant, even the shortened version 4.

If we were to go with variant 1 however, I think it would add more to have a link to the pgbench documentation that refers to the different init steps. Perhaps on a new line just under where it says "see pgbench documentation for a description of these steps".

Overall good patch, I'm a firm believer that more information is always better than less.

Tristen
---------------
Software Engineer
HighGo Software Inc. (Canada)
tristen(dot)raab(at)highgo(dot)ca
www.highgo.ca

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message cary huang 2023-09-22 21:46:41 Re: Add support for AT LOCAL
Previous Message Jeff Davis 2023-09-22 20:52:48 Is this a problem in GenericXLogFinish()?