Re: Long options for pg_ctl waiting

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

In response to

Responses

Browse pgsql-hackers by date

  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