Hi,

I rebased and updated the patch to address some concerns raised before and
see if anyone is still interested in this.

I believe that there is a general consensus around RelCacheContext and
CatCacheContext, considering that these two caches are fairly used. For the
rest, I followed Ashutosh's suggestion [1] and kept Get*CacheContext()
functions. But those functions do not create a separate context, simply use
CacheMemoryContext instead. It'd be easier to change those to actually
create new memory contexts, if it's decided to have more granularity in
CacheMemoryContext in the future.

What are your thoughts?


[1]
https://www.postgresql.org/message-id/CAExHW5sH4NZnHi4S5ai0uFQgfS_R=rts_+lk5jeeq-dvzwk...@mail.gmail.com

Thanks,
-- 
Melih Mutlu
Microsoft

Attachment: v5-0001-Separate-memory-contexts-for-caches.patch
Description: Binary data

Attachment: v5-0002-Adjusting-cache-memory-context-sizes.patch
Description: Binary data

Reply via email to