From: | Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: Get rid of the need for manual maintenance of the initial |
Date: | 2010-01-05 19:23:15 |
Message-ID: | 4B4391A3.2090203@kaltenbrunner.cc |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc> writes:
>> did that and it seems the problem is in the loop that does:
>
>> foreach my $row (@$data)
>> {
>
>> # To construct fmgroids.h and fmgrtab.c, we need to inspect some
>> # of the individual data fields. Just splitting on whitespace
>
> Huh. It's weird that I don't see a leak in either 5.8.7 or 5.10.1,
> which are the two closest perl versions I have handy here. I think
> this may be a platform-specific Perl bug. Still, it would be nice
> to work around it if we can.
yeah it is probably some strange platform specific issue - however with
some help from the IRC channel I came up with the following patch that
considerable reduces the memory bloat (but still needs ~55MB max) and
allows the build to succeed.
>
> I'm not nearly good enough in Perl to be sure about the semantics
> of this loop. Is it possible that it's changing the global contents
> of the @$data structure, rather than just hacking a local copy of
> each row before pushing some values into @fmgr?
hmm it is leaking a constant amount of 240kbyte per loop iteration with
the original code but yeah very weird issue...
Stefan
Attachment | Content-Type | Size |
---|---|---|
gen_fmgrtab_workaround.patch | text/x-patch | 559 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2010-01-05 19:31:43 | Re: Re: [COMMITTERS] pgsql: Get rid of the need for manual maintenance of the initial |
Previous Message | Alvaro Herrera | 2010-01-05 19:04:05 | Re: Re: [COMMITTERS] pgsql: Get rid of the need for manual maintenance of the initial |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2010-01-05 19:31:43 | Re: Re: [COMMITTERS] pgsql: Get rid of the need for manual maintenance of the initial |
Previous Message | Peter Eisentraut | 2010-01-05 19:21:16 | Re: libpq naming on Win64 |