Re: pgsql: Make new strings more alike previously existing messages.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Make new strings more alike previously existing messages.
Date: 2009-04-13 22:57:00
Message-ID: 24097.1239663420@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

alvherre(at)postgresql(dot)org (Alvaro Herrera) writes:
> Make new strings more alike previously existing messages.

This patch changed the meaning of one of the messages:

*************** main(int argc, char **argv)
*** 316,322 ****
/* Can't do single-txn mode with multiple connections */
if (opts->single_txn && opts->number_of_jobs > 1)
{
! fprintf(stderr, _("%s: cannot specify both --single-transaction and multiple jobs\n"),
progname);
exit(1);
}
--- 316,322 ----
/* Can't do single-txn mode with multiple connections */
if (opts->single_txn && opts->number_of_jobs > 1)
{
! fprintf(stderr, _("%s: options -1/--single-transaction and -j/--jobs cannot be used together\n"),
progname);
exit(1);
}

As the comment and the code itself show, "-j 1" works just fine with
--single-transaction. The original wording reflected that, the new does
not.

Maybe this amount of imprecision is acceptable, but "make it look more
like the other messages" doesn't seem to me like enough justification.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2009-04-14 00:06:34 Re: pgsql: Make new strings more alike previously existing messages.
Previous Message Alvaro Herrera 2009-04-13 21:03:36 pgsql: Make new strings more alike previously existing messages.