From: | Vik Fearing <vik(at)2ndquadrant(dot)fr> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Gavin Flower <GavinFlower(at)archidevsys(dot)co(dot)nz>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Long options for pg_ctl waiting |
Date: | 2016-09-07 23:46:03 |
Message-ID: | 1607d014-3de8-cb05-a2d6-7645ac40c1f3@2ndquadrant.fr |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 09/08/2016 01:05 AM, Tom Lane wrote:
> Vik Fearing <vik(at)2ndquadrant(dot)fr> writes:
>> On 09/07/2016 11:39 PM, Gavin Flower wrote:
>>> Possibly generate warningswhen the non hyphenated form is used?
>
>> I'm not quite sure how I got volunteered to do this work, but it's easy
>> enough so I don't mind.
>> Here is a new patch that emits a warning when --noclean and/or --nosync
>> are used.
>
> I'm pretty much -1 on printing a warning. There's no ambiguity, and no
> real reason for us ever to remove the old spellings. Standardizing on
> "no-" going forward makes sense, but let's not slap people's wrists for
> existing usage. (Or: if it ain't broke, don't break it.)
One could also argue that 2 out of 53 "no" options omitting the dash is
in fact broken, and a real reason to remove them.
I don't see the warning as "slapping wrists" so much as saying that
we're harmonizing our conventions and their scripts need to be updated
for the better.
That said, I'm not going to fight for this. My only goal here is to get
--wait and --no-wait added to pg_ctl.
--
Vik Fearing +33 6 46 75 15 36
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-09-07 23:56:31 | Re: Long options for pg_ctl waiting |
Previous Message | Robins Tharakan | 2016-09-07 23:37:31 | High-CPU consumption on information_schema (only) query |