Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regressiontest - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regressiontest
Date
Msg-id 243e65e5-f928-971a-f764-52acac2f3acb@2ndQuadrant.com
Whole thread Raw
In response to Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regressiontest  (Andrew Dunstan <andrew.dunstan@2ndquadrant.com>)
Responses Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regression test  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers

On 05/14/2017 12:04 AM, Andrew Dunstan wrote:
>
> On 05/13/2017 11:32 PM, Robert Haas wrote:
>> On Sat, May 13, 2017 at 6:22 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> Or at least, that's what I surmise from the fact that buildfarm critter
>>> caiman has been failing that test for the last day or so, with symptoms
>>> indicating whitespace changes in Data::Dumper output.  Some poking into
>>> the Fedora repo shows that rawhide updated perl-Data-Dumper from 2.161
>>> to 2.167 on May 11, so that fits ...
>> Depending on the precise details of how Data::Dumper prints things
>> doesn't seem like a particularly good idea.
>>
>
> I'd be inclined to set $Data::Dumper::Indent to 0 which would suppress
> all indentation, and adjusting the test results accordingly. We already
> set $Data::Dumper::Sortkeys to 1, so there's precedent for controlling
> how it presents data back to us.
>


Here's a patch along those lines.

cheers

andrew


-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regression test
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] Latest Data::Dumper breaks hstore_plperl regression test