From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Removed extra memory allocations from create_list_bounds |
Date: | 2021-05-16 17:00:29 |
Message-ID: | 20210516170029.GB373@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, May 15, 2021 at 02:40:45PM +0530, Nitin Jadhav wrote:
> While working on [1], I observed that extra memory is allocated in
> [1] https://mail.google.com/mail/u/2/#search/multi+column+list/KtbxLxgZZTjRxNrBWvmHzDTHXCHLssSprg?compose=CllgCHrjDqKgWCBNMmLqhzKhmrvHhSRlRVZxPCVcLkLmFQwrccpTpqLNgbWqKkTkTFCHMtZjWnV
This is a link to your gmail, not to anything public.
If it's worth counting list elements in advance, then you can also allocate the
PartitionListValue as a single chunk, rather than palloc in a loop.
This may help large partition heirarchies.
And the same thing in create_hash_bounds with hbounds.
create_range_bounds() already doesn't call palloc in a loop. However, then
there's an asymmetry in create_range_bounds, which is still takes a
double-indirect pointer.
I'm not able to detect that this is saving more than about ~1% less RAM, to
create or select from 1000 partitions, probably because other data structures
are using much more, and savings here are relatively small.
I'm going to add to the next CF. You can add yourself as an author, and watch
that the patch passes tests in cfbot.
https://commitfest.postgresql.org/
http://cfbot.cputube.org/
Thanks,
--
Justin
Attachment | Content-Type | Size |
---|---|---|
0001-Removed-extra-memory-allocations-from-create_list_bo.patch | text/x-diff | 3.5 KB |
0002-allocate-the-PartitionListValue-as-a-single-chunk.patch | text/x-diff | 2.9 KB |
0003-Same-for-create_hash_bounds-PartitionHashBound.patch | text/x-diff | 3.3 KB |
0004-Allocate-datum-arrays-in-bulk-to-avoid-palloc-overhe.patch | text/x-diff | 3.1 KB |
0005-create_range_bounds-pfree-intermediate-results.patch | text/x-diff | 951 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-05-16 17:04:15 | Re: Query about time zone patterns in to_char |
Previous Message | Nitin Jadhav | 2021-05-16 16:13:21 | Re: Query about time zone patterns in to_char |