Re: [PATCH] get rid of StdRdOptions, use individual binaryreloptions representation for each relation kind instead

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: [PATCH] get rid of StdRdOptions, use individual binaryreloptions representation for each relation kind instead
Дата
Msg-id 20180302001532.irqf4nwegpezqitt@alap3.anarazel.de
обсуждение исходный текст
Ответ на [PATCH] get rid of StdRdOptions, use individual binary reloptions representation for each relation kind instead  (Nikolay Shaplov <dhyan@nataraj.su>)
Ответы Re: [PATCH] get rid of StdRdOptions, use individual binary reloptions representation for each relation kind instead  (Nikolay Shaplov <dhyan@nataraj.su>)
Список pgsql-hackers
Hi,

On 2018-02-22 19:48:46 +0300, Nikolay Shaplov wrote:
> This is part or my bigger patch
https://www.postgresql.org/message-id/flat/2146419.veIEZdk4E4@x200m#2146419.veIEZdk4E4@x200mwe've decided to 
 
> commit by smaller parts.

I've not read that thread. Is this supposed to be a first step towards
something larger?


> So for example if you set custom fillfactor value for some index, then it will 
> lead to allocating 84 bytes of memory (sizeof StdRdOptions on i386) and only 8 
> bytes will be actually used (varlena header and fillfactor value). 74 bytes is 
> not much, but allocating them for each index for no particular reason is bad 
> idea, as I think.

I'm not sure this is a particularly strong motivator though?  Does the
patch have a goal besides saving a few bytes?


> Moreover when I wrote my big reloption refactoring patch, I came to "one 
> reloption kind - one binary representation" philosophy. It allows to write 
> code with more logic in it.

I have no idea what this means?


Are you aiming this for v11 or v12?


Greetings,

Andres Freund


В списке pgsql-hackers по дате отправления:

Предыдущее
От: David Steele
Дата:
Сообщение: Re: Hash Joins vs. Bloom Filters / take 2
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Rewrite of pg_dump TAP tests