Re: BUG #12379: pgbench should hint to pgbench -i

From: Julien Rouhaud <julien(dot)rouhaud(at)dalibo(dot)com>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Oleksandr Shulgin <oleksandr(dot)shulgin(at)zalando(dot)de>
Cc: mail(at)bwe(dot)im, pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #12379: pgbench should hint to pgbench -i
Date: 2015-07-04 20:21:26
Message-ID: 55984046.60901@dalibo.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On 22/05/2015 11:33, Fabien COELHO wrote:
>
>>> HINT: is "foo" the right database? did you initialize first (pgbench
>>> -i)?
>>
>> I don't think there is precedent for spitting out "HINT:" in client code
>> (at least grep didn't find it). So this message might look like it's
>> coming from the server, while it doesn't.
>
> Here is a v3, which makes it clearer that the hint comes from pgbench.
>
> HINT pgbench: is "foo" the right database? did you initialize first (-i)?
>

Hello,

I've just reviewed this patch. First, I've attached a rebased version
(v4), as the v3 doesn't apply anymore.

The patch is pretty simple, so everything is ok.

But I've noticed that if you don't specify any database to pgbench, you
get an empty database information:

HINT pgbench: is "" the right database? did you initialize
first (-i)?

This isn't related to this patch specifically, but it may be confusing
for users. At least more than the other places it could already appear
(debug mode and failed connection attempt).

As the database name is optional, wouldn't it be better to fix this
behavior?

Regards.

--
Julien Rouhaud
http://dalibo.com - http://dalibo.org

Attachment Content-Type Size
pgbench-hint-4.patch text/x-patch 479 bytes

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Guillaume Lelarge 2015-07-05 06:57:02 Re: BUG #12379: pgbench should hint to pgbench -i
Previous Message jingwei_5107 2015-07-04 15:12:03 BUG #13487: GetBufferFromRing code bug