From: | Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com> |
---|---|
To: | torikoshia <torikoshia(at)oss(dot)nttdata(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, andres(at)anarazel(dot)de |
Subject: | Re: Separate memory contexts for relcache and catcache |
Date: | 2024-01-03 11:26:34 |
Message-ID: | CAGPVpCQqAbBNaFQ7YiTnF5ug2uGxK-HQBMofh1QT9JQQB3Bu8g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>, 4 Ara 2023 Pzt, 07:59 tarihinde
şunu yazdı:
> Hi,
>
> I also think this change would be helpful.
>
> I imagine you're working on the Andres's comments and you already notice
> this, but v1 patch cannot be applied to HEAD.
> For the convenience of other reviewers, I marked it 'Waiting on Author'.
>
Thanks for letting me know. I rebased the patch. PFA new version.
Andres Freund <andres(at)anarazel(dot)de>, 12 Eki 2023 Per, 20:01 tarihinde şunu
yazdı:
> Hi,
>
> Have you checked what the source of the remaining allocations in
> CacheMemoryContext are?
>
It's mostly typecache, around 2K. Do you think typecache also needs a
separate context?
Given the size of both CatCacheMemoryContext and RelCacheMemoryContext in a
> new backend, I think it might be worth using non-default aset parameters. A
> bit ridiculous to increase block sizes from 8k upwards in every single
> connection made to postgres ever.
>
Considering it starts from ~262K, what would be better for init size?
256K?
> +static void
> > +CreateCatCacheMemoryContext()
>
> We typically use (void) to differentiate from an older way of function
> declarations that didn't have argument types.
>
Done.
> +{
> > + if (!CacheMemoryContext)
> > + CreateCacheMemoryContext();
>
> I wish we just made sure that cache memory context were created in the
> right
> place, instead of spreading this check everywhere...
>
That would be nice. Do you have a suggestion about where that right place
would be?
I'd just delete these comments, they're just pointlessly restating the code.
>
Done.
Thanks,
--
Melih Mutlu
Microsoft
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Separate-memory-contexts-for-relcache-and-catcach.patch | application/octet-stream | 17.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bertrand Drouvot | 2024-01-03 11:26:57 | Re: Synchronizing slots from primary to standby |
Previous Message | jian he | 2024-01-03 10:53:34 | Re: remaining sql/json patches |