Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-13 Thread Tom Lane
David Fetter writes: > On Tue, Aug 13, 2019 at 03:08:34PM +0530, amul sul wrote: >> Look like commit # d3f48dfae42 added the required fix but is enabled only >> for the clobber-cache builds :( > I've got a real world multi-tenancy case that would really be helped > by this. Can we enable it for a

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-13 Thread David Fetter
On Tue, Aug 13, 2019 at 03:08:34PM +0530, amul sul wrote: > On Sat, Aug 10, 2019 at 12:16 AM David Fetter wrote: > > > On Thu, Aug 08, 2019 at 05:42:21PM +0900, Amit Langote wrote: > > > On Thu, Aug 8, 2019 at 5:33 PM amul sul wrote: > > > > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote > > wrote

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-13 Thread amul sul
On Sat, Aug 10, 2019 at 12:16 AM David Fetter wrote: > On Thu, Aug 08, 2019 at 05:42:21PM +0900, Amit Langote wrote: > > On Thu, Aug 8, 2019 at 5:33 PM amul sul wrote: > > > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote > wrote: > > > > >> Thanks for the patch. This was discussed recently in the

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-09 Thread David Fetter
On Thu, Aug 08, 2019 at 05:42:21PM +0900, Amit Langote wrote: > On Thu, Aug 8, 2019 at 5:33 PM amul sul wrote: > > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote wrote: > > >> Thanks for the patch. This was discussed recently in the "hyrax vs. > >> RelationBuildPartitionDesc()" thread [1] and I th

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-08 Thread Amit Langote
On Thu, Aug 8, 2019 at 5:33 PM amul sul wrote: > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote 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 wh

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-08 Thread amul sul
On Thu, Aug 8, 2019 at 1:27 PM Amit Langote wrote: > Hi Amul, > > On Thu, Aug 8, 2019 at 4:15 PM amul sul wrote: > > > > Hi, > > > > In RelationBuildPartitionDesc(), a memory space that use to gather > partitioning > > bound info wasn't free at the end. This might not a problem because this > >

Re: Some memory not freed at the exit of RelationBuildPartitionDesc()

2019-08-08 Thread Amit Langote
Hi Amul, On Thu, Aug 8, 2019 at 4:15 PM amul sul wrote: > > Hi, > > In RelationBuildPartitionDesc(), a memory space that use to gather > partitioning > bound info wasn't free at the end. This might not a problem because this > allocated memory will eventually be recovered when the top-level con