From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Lætitia Avrot <laetitia(dot)avrot(at)gmail(dot)com> |
Subject: | Re: pgsql: Add support for hyperbolic functions, as well as log10(). |
Date: | 2019-03-14 02:44:30 |
Message-ID: | 1225.1552531470@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Wed, Mar 13, 2019 at 10:39 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> No, but that's not the hazard. I have a very fresh-in-mind example:
>> at one point while tweaking Laetitia's patch, I'd accidentally changed
>> datanh so that it called tanh not atanh. The previous set of tests did
>> not reveal that :-(
> Well, that was a goof, but it's not likely that such a regression will
> ever be reintroduced.
Sure, but how about this for another example: maybe a given platform
hasn't got these functions (or they're in a different library we
didn't pull in), but you don't see a failure until you actually
call them. We try to set up our link options so that that sort
of failure is reported at build time, but I wouldn't care to bet
that we've succeeded at that everywhere.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2019-03-14 03:06:51 | Re: pgsql: Add support for hyperbolic functions, as well as log10(). |
Previous Message | Robert Haas | 2019-03-14 02:40:18 | Re: pgsql: Add support for hyperbolic functions, as well as log10(). |
From | Date | Subject | |
---|---|---|---|
Next Message | Kyotaro HORIGUCHI | 2019-03-14 02:54:17 | Re: Progress reporting for pg_verify_checksums |
Previous Message | Robert Haas | 2019-03-14 02:40:18 | Re: pgsql: Add support for hyperbolic functions, as well as log10(). |