Hello Ashutosh, Thank you for your detailed review, and apologies for my delayed response.
On Thu, Mar 27, 2025 at 1:42 PM Ashutosh Bapat <ashutosh.bapat....@gmail.com> wrote: > > Here are memory consumption numbers using list_free() instead of pfree(), > using the same method as [1], using a binary without asserts and debug info. > PFA the patchset where all the patches are the same as v35 but with an extra > patch fixing memory leak. The memory leak is visible with a higher number of > joins. At a lower number of joins, I expect that the memory saved is less > than a KB or the leaked memory fits within 1 chunk of memory context and > hence not visible. Thank you for conducting your benchmarks. Your results clearly show increased memory consumption with my patches. As Tom also suggested, we may reduce memory usage by adopting a different design. I will reconsider alternative approaches and compare the memory usage to the current version. Thank you once again for conducting the benchmarks. -- Best regards, Yuya Watari
