Re: [PATCH] Cleanup of GUC units code - Mailing list pgsql-hackers

From Hannu Krosing
Subject Re: [PATCH] Cleanup of GUC units code
Date
Msg-id 1220455936.12162.20.camel@huvostro
Whole thread Raw
In response to Re: [PATCH] Cleanup of GUC units code  ("Joshua D. Drake" <jd@commandprompt.com>)
Responses Re: [PATCH] Cleanup of GUC units code
List pgsql-hackers
On Wed, 2008-09-03 at 07:52 -0700, Joshua D. Drake wrote:
> Hannu Krosing wrote:
> > On Tue, 2008-09-02 at 16:50 +0300, Peter Eisentraut wrote:
> >> Gregory Stark wrote:
> >>> Peter Eisentraut <peter_e@gmx.net> writes:
> 
> > Are you really afraid that someone would want to use mb to mean
> > millibits ?
> > 
> > As SQL is generally case insensitive, it is quite surprising to most
> > people that GUC units are not.
> 
> We have had this discussion before, I even submitted a patch to make 
> them case insensitive. In retrospect I was wrong to submit that patch. 
> SQL may be case insensitive but units are not. MB != Mb != mb , 

For most people they are equal, and all mean MEGABYTE(S) though
http://en.wikipedia.org/wiki/MB has lots of other possible meanings for
each.

> I don't think we should encourage in any way for users to do the wrong thing.

Can you see any scenario where accepting case insensitive units does
more damage than just ignoring the conf line with "incorrect" casing ?

Or do you mean we should discourage people from editing postgresql.conf
manually and have them use some tool which prevents them entering "kb" ?

----------------
Hannu




pgsql-hackers by date:

Previous
From: "Erikj"
Date:
Subject: Window functions doc patch
Next
From: Hannu Krosing
Date:
Subject: Re: [PATCH] Cleanup of GUC units code