From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | Liam Stewart <liams(at)redhat(dot)com>, PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCHES] guc |
Date: | 2002-01-17 22:57:29 |
Message-ID: | 25748.1011308249@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Liam Stewart writes:
>> I've removed
>> GetConfigOption() and replaced it with ShowConfigOption() which does an
>> elog(NOTICE) instead of returning a string.
> I certainly don't like that. I want to be able to get at the
> configuration setting without any notice going off.
I agree with Peter --- better to separate the getting of the string
from displaying it. What's the reason for collapsing them together?
>> When setting REAL_FORMAT and DOUBLE_PRECISION_FORMAT, the printf-style
>> format string should be checked for certain things:
> I wouldn't use a printf format string at all.
Good point. If we have to set up a checking mechanism then we should
ask ourselves why we're bothering to use printf representation.
> What I would like to get out of the configurability of floating-point
> numbers is:
> 1. The ability to dump them in binary or hex format for lossless
> dump/reload. (printf("%a") does that.)
On some platforms... I'd be happier with this if it were more portable...
> Some people will also suggest
> 3. Be able to set the number of significant digits that are shown.
> to allow simplifying the regression tests, but I do not think that that is
> a good idea,
[ raises eyebrow ] To my mind that was one of the principal reasons
for working on such a thing at all. If you don't want to allow this,
then what alternative solution do you have for our geometry regression
test mess? (Defining it as not a mess won't fly.)
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2002-01-17 23:12:21 | Re: [PATCHES] guc |
Previous Message | Peter Eisentraut | 2002-01-17 22:35:24 | Re: [PATCHES] guc |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2002-01-17 23:07:20 | Re: postgresql-7.2b3-betterquote.patch |
Previous Message | Tom Lane | 2002-01-17 22:36:42 | Re: postgresql-7.2b3-betterquote.patch |