From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Reduce TupleHashEntryData struct size by half |
Date: | 2025-03-25 06:14:43 |
Message-ID: | ee3ae05c7a2cf166d75c01508185ad093ca693be.camel@j-davis.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, 2025-03-22 at 09:39 -0700, Jeff Davis wrote:
> For some reason I'm getting a decline of about 3% in the c.sql test
> that seems to be associated with the accessor functions, even when
> inlined. I'm also not seeing as much benefit from the inlining of the
> MemoryContextMemAllocated(). But these mixed test results are minor
> compared with the memory savings of 35% and the more consistently-
> improved performance of 5% on the larger test (which is also
> integers),
> so I plan to commit it.
Committed, except for v9-0007. (Note that some commits were combined;
they were only separated originally for performance testing.)
I attached a new version of v9-0007, now v10-0001, that uses
recurse=false for two of the memory contexts. I didn't see a major
speedup, but posting here anyway.
Regards,
Jeff Davis
Attachment | Content-Type | Size |
---|---|---|
v10-0001-Inline-MemoryContextMemAllocated.patch | text/x-patch | 7.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2025-03-25 06:16:53 | Re: Reducing memory consumed by RestrictInfo list translations in partitionwise join planning |
Previous Message | Alena Rybakina | 2025-03-25 06:12:57 | Re: Vacuum statistics |