Re: RFC: list API / memory allocations
От | Stephen Frost |
---|---|
Тема | Re: RFC: list API / memory allocations |
Дата | |
Msg-id | 20111119173109.GZ24234@tamriel.snowman.net обсуждение исходный текст |
Ответ на | RFC: list API / memory allocations (Andres Freund <andres@anarazel.de>) |
Ответы |
Re: RFC: list API / memory allocations
Re: RFC: list API / memory allocations |
Список | pgsql-hackers |
Andres, * Andres Freund (andres@anarazel.de) wrote: > For that I added new functions/defines which allocate all the needed memory in > one hunk: > list_immutable_make$n(), > List *list_new_immutable_n(NodeTag t, size_t n); > List *list_make_n(NodeTag t, size_t n, ...); A while back, I posted a patch to try and address this same issue. The approach that I took was to always pre-allocate a certain (#defined) amount (think it was 5 or 10 elements). There were a number of places that caused problems with that approach because they hacked on the list element structures directly (instead of using the macros/functions)- you'll want to watch out for those areas in any work on lists. That patch is here: http://archives.postgresql.org/pgsql-hackers/2011-05/msg01213.php The thread on it might also be informative. I do like your approach of being able to pass the ultimate size of the list in.. Perhaps the two approaches could be merged? I was able to make everything work with my approach, provided all the callers used the list API (I did that by making sure the links, etc, actually pointed to the right places in the pre-allocated array). One downside was that the size ended up being larger that it might have been in some cases. Thanks, Stephen
В списке pgsql-hackers по дате отправления: