Re: float8 regression failure (HEAD, cygwin)

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Adrian Maier <adrian(dot)maier(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: float8 regression failure (HEAD, cygwin)
Date: 2006-08-01 12:57:40
Message-ID: 44CF4FC4.7070802@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Maybe we need to abandon trying to map float8 results exactly in the
>> resultmap file, and just let pg_regress pick the best fit as we do with
>> some other tests.
>>
>
> I thought about that too but it seems a very bad idea. small-is-zero is
> distinctly "less correct" than the regular output, and I don't think we
> want pg_regress to be blindly accepting it as OK on any platform.
>
> Perhaps we could stick a version check into the resultmap lookup? It'd
> likely have been painful on the shell script implementation but now that
> the code is in C I think we have lots of flexibility. There's no need
> to feel bound by the historical resultmap format.
>
> However this is all premature unless we can verify that "cgywin's strtod()
> complains about float underflow after version so-and-so". Do they
> publish a detailed change log?
>
>
>

Yes, good points. One other thought I had was that we could have
pg_regress always allow a fallback to the canonical result file. So in
this case it would try float8-small-is-zero.out and
float8-small-is-zero_1.out and finally float8.out (the first would
succeed on eel, the last on cassowary, I am speculating). In general
this would allow a configuration to become more correct painlessly.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-08-01 13:01:53 Re: [HACKERS] 8.2 features?
Previous Message Adrian Maier 2006-08-01 12:45:08 Re: [HACKERS] float8 regression failure (HEAD, cygwin)

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-08-01 13:01:53 Re: [HACKERS] 8.2 features?
Previous Message Adrian Maier 2006-08-01 12:45:08 Re: [HACKERS] float8 regression failure (HEAD, cygwin)