From: | Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Noah Misch <noah(at)leadboat(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Geoghegan <pg(at)bowt(dot)ie>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>, Stephen Frost <sfrost(at)snowman(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, Amul Sul <sulamul(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_amcheck contrib application |
Date: | 2021-03-13 06:33:29 |
Message-ID: | 55AC44F2-D4D6-4687-B172-2D48F647DDA1@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On Mar 12, 2021, at 10:28 PM, Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com> wrote:
>
>
>
>> On Mar 12, 2021, at 10:22 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>
>> Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com> writes:
>>> On Mar 12, 2021, at 10:16 PM, Noah Misch <noah(at)leadboat(dot)com> wrote:
>>>> hoverfly does configure with PERL=perl64. /usr/bin/prove is from the 32-bit
>>>> Perl, so I suspect the TAP suites get 32-bit Perl that way. (There's no
>>>> "prove64".)
>>
>> Oh, that's annoying.
>>
>>>> This test should unpack the field as two 32-bit values, not a
>>>> 64-bit value, to avoid requiring more from the Perl installation.
>>
>>> I will post a modified test in a bit that avoids using Q/q.
>>
>> Coping with both endiannesses might be painful.
>
> Not too bad if the bigint value is zero, as both the low and high 32bits will be zero, regardless of endianness. The question is whether that gives up too much in terms of what the test is trying to do. I'm not sure that it does, but if you'd rather solve this by upgrading perl, that's ok by me.
I'm not advocating that this be the solution, but if we don't fix up the perl end of it, then this test change might be used instead.
Attachment | Content-Type | Size |
---|---|---|
v5-0001-pg_amcheck-continuing-to-fix-portability-problems.patch | application/octet-stream | 3.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-03-13 06:36:11 | Re: pg_amcheck contrib application |
Previous Message | Tom Lane | 2021-03-13 06:31:39 | Re: pgbench: option delaying queries till connections establishment? |