Re: pgsql: Avoid using platform-dependent floats in test case. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject Re: pgsql: Avoid using platform-dependent floats in test case.
Date
Msg-id be031813-bd4b-4b02-7012-01fd37894410@iki.fi
Whole thread Raw
In response to Re: pgsql: Avoid using platform-dependent floats in test case.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On 10/26/2016 04:42 PM, Tom Lane wrote:
> Heikki Linnakangas <heikki.linnakangas@iki.fi> writes:
>> Avoid using platform-dependent floats in test case.
>> The number of decimals printed for floats varied in this test case, as
>> noted by several buildfarm members. There's nothing special about floats
>> and arrays in the code being tested, so replace the floats with numerics to
>> make the output platform-independent.
>
> Numerics aren't entirely platform-independent in Python, either,
> cf prairiedog.  Is it okay to use text?

Oh, thanks for prodding, I didn't notice it was still failing, I thought
it just hadn't run since that fix.

There is actually a similar test for text already. I don't really have
any reason to believe that this would work differently with different
datatypes, adding tests with different datatypes was perhaps overkill in
the first place. I'm going to just remove the offending case.

- Heikki



pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgsql: Avoid using platform-dependent floats in test case.
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Remove platform-dependent PL/python test case.