pgsql: Rethink how to test the hyperbolic functions.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Rethink how to test the hyperbolic functions.
Date: 2019-03-13 22:14:05
Message-ID: E1h4C8T-0007tQ-W2@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Rethink how to test the hyperbolic functions.

The initial commit tried to test them on trivial cases such as 0,
reasoning that we shouldn't hit any portability issues that way.
The buildfarm immediately proved that hope ill-founded, and anyway
it's not a great testing scheme because it doesn't prove that we're
even calling the right library function for each SQL function.

Instead, let's test them at inputs such as 1 (or something within
the valid range, as needed), so that each function should produce
a different output.

As committed, this is just about certain to show portability
failures, because it's very unlikely that every platform computes
these functions the same as mine down to the last bit. However,
I want to put it through a buildfarm cycle this way, so that
we can see how big the variations are. The plan is to add
"set extra_float_digits = -1", or whatever we need in order to
hide the variations; but first we need data.

Discussion: https://postgr.es/m/E1h3nUY-0000sM-Vf@gemulon.postgresql.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c6f153dcfebccf7a0d92290037793c656f1caef5

Modified Files
--------------
src/test/regress/expected/float8.out | 50 ++++++++++++++++++------------------
src/test/regress/sql/float8.sql | 16 ++++++------
2 files changed, 33 insertions(+), 33 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2019-03-13 23:02:24 Re: pgsql: Use condition variables to wait for checkpoints.
Previous Message Thomas Munro 2019-03-13 22:02:43 pgsql: Use condition variables to wait for checkpoints.