Re: Some memory not freed at the exit of RelationBuildPartitionDesc() - Mailing list pgsql-hackers

From David Fetter
Subject Re: Some memory not freed at the exit of RelationBuildPartitionDesc()
Date
Msg-id 20190809184609.GU31493@fetter.org
Whole thread Raw
In response to Re: Some memory not freed at the exit of RelationBuildPartitionDesc()  (Amit Langote <amitlangote09@gmail.com>)
Responses Re: Some memory not freed at the exit of RelationBuildPartitionDesc()
List pgsql-hackers
On Thu, Aug 08, 2019 at 05:42:21PM +0900, Amit Langote wrote:
> On Thu, Aug 8, 2019 at 5:33 PM amul sul <sulamul@gmail.com> wrote:
> > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote <amitlangote09@gmail.com> wrote:
> 
> >> Thanks for the patch.  This was discussed recently in the "hyrax vs.
> >> RelationBuildPartitionDesc()" thread [1] and I think Alvaro proposed
> >> an approach that's similar to yours.  Not sure why it wasn't pursued
> >> though.  Maybe the reason is buried somewhere in that discussion.
> >
> > Oh, quite similar, thanks Amit for pointing that out.
> >
> > Look like "hyrax vs.RelationBuildPartitionDesc()" is in discussion for the master
> > branch only, not sure though, but we need the similar fix for the back branches as well.
> 
> Well, this is not a bug as such, so it's very unlikely that a fix like
> this will be back-patched.  Also, if this becomes an issue only for
> more than over 1000 partitions, then it's not very relevant for PG 10
> and PG 11, because we don't recommend using so many partitions with
> them.  Maybe we can consider fixing PG 12 though.

A fix for the thousands-of-partitions case would be very welcome for 12.

Best,
David.
-- 
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate



pgsql-hackers by date:

Previous
From: Jeevan Ladhe
Date:
Subject: Re: block-level incremental backup
Next
From: "Jonathan S. Katz"
Date:
Subject: Re: Add "password_protocol" connection parameter to libpq