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

From Alvaro Herrera
Subject Re: [PATCH] Cleanup of GUC units code
Date
Msg-id 20080908143240.GD4411@alvh.no-ip.org
Whole thread Raw
In response to Re: [PATCH] Cleanup of GUC units code  (Gregory Stark <stark@enterprisedb.com>)
Responses Re: [PATCH] Cleanup of GUC units code  (Gregory Stark <stark@enterprisedb.com>)
Re: [PATCH] Cleanup of GUC units code  (Joshua Drake <jd@commandprompt.com>)
List pgsql-hackers
Gregory Stark wrote:
> "Greg Sabino Mullane" <greg@turnstep.com> writes:
> 
> > Tom Lane wrote:
> >> My vote is to reject the patch and work on a config checker.
> >
> > +1
> >
> >> postgres=# set work_mem = '1g';
> >> ERROR: invalid value for parameter "work_mem": "1g"
> >
> > Perhaps this would be a great place for a HINT listing all
> > valid inputs, if not there already?
> 
> It is, I paraphrased it on my original message as:
> 
> HINT:  It's perfectly clear what you want but I'm going to refuse to do
>        it until you type it exactly as I say: "GB"

It's good as a joke, but what if the user says '1024b'?  Does it mean
1024 blocks or one kilobyte?  If blocks, what size are we talking, the
usual 512 bytes, or our BLCKSZ?

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


pgsql-hackers by date:

Previous
From: Gregory Stark
Date:
Subject: Re: [PATCH] Cleanup of GUC units code
Next
From: Gregory Stark
Date:
Subject: Re: [PATCH] Cleanup of GUC units code