Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line - Mailing list pgsql-hackers

From Amit Langote
Subject Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line
Date
Msg-id e720e262-2c68-c176-c3a4-22975b6974a7@lab.ntt.co.jp
Whole thread Raw
In response to Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line  (Jeff Janes <jeff.janes@gmail.com>)
Responses Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes there should be per DATA() line  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 2017/03/10 9:14, Jeff Janes wrote:
> On Thu, Mar 9, 2017 at 3:20 PM, Robert Haas <robertmhaas@gmail.com
> <mailto:robertmhaas@gmail.com>> wrote:
> 
>     On Mon, Mar 6, 2017 at 11:37 AM, Dagfinn Ilmari Mannsåker
>     <ilmari@ilmari.org <mailto:ilmari@ilmari.org>> wrote:
>     > David Christensen <david@endpoint.com <mailto:david@endpoint.com>> writes:
>     >>> Hi David,
>     >>>
>     >>> Here's a review of your patch.
>     >>
>     >> Hi Ilmari, thanks for your time and review.  I’m fine with the revised version.
>     >
>     > Okay, I've marked the patch as Ready For Committer.
> 
>     Committed.   Hopefully this doesn't contain any Perl bits that are
>     sufficiently new as to cause problems for our older BF members ... I
>     guess we'll see.
> 
> 
> 
> Bad luck there.  I'm getting this error on CentOS6.8, perl v5.10.1
>
> Can't locate object method "input_line_number" via package "IO::Handle" at
> ../../../src/backend/catalog/Catalog.pm line 82, <INPUT_FILE> line 148.
> make[3]: *** [fmgrtab.c] Error 25
> make[2]: *** [utils/fmgroids.h] Error 2
> make[2]: *** Waiting for unfinished jobs....
> Can't locate object method "input_line_number" via package "IO::Handle" at
> ../../../src/backend/catalog/Catalog.pm line 82, <INPUT_FILE> line 148.
> make[3]: *** [postgres.bki] Error 25
> make[2]: *** [submake-schemapg] Error 2
> make[1]: *** [all-backend-recurse] Error 2
> make: *** [all-src-recurse] Error 2

Same here.  Some buildfarm animals failed too.

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=dromedary&dt=2017-03-10%2000%3A55%3A42

> I think we can just save $. and use that, as in the attached.

The patch works for me.

Thanks,
Amit





pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] Re: [COMMITTERS] pgsql: Throw an error if a DATA() line contains wrong # of attributes.
Next
From: Stephen Frost
Date:
Subject: Re: [HACKERS] GSOC Introduction / Eliminate O(N^2) scaling fromrw-conflict tracking in serializable transactions